Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NAS-133758 / 25.04-RC.1 / Add support for randomizing user passwords (by anodos325) #15499

Merged
merged 1 commit into from
Jan 28, 2025

Conversation

bugclerk
Copy link
Contributor

This commit adds support for randomizing the user password for new users. This is to fulfill backend expectations of password being present while admin provides a one-time password to log into the server and perform operations.

Note: this commit changes the output of user.create and user.update so that it returns a copy of the
entry instead of just returning the pk.

Original PR: #15469
Jira URL: https://ixsystems.atlassian.net/browse/NAS-133758

This commit adds support for randomizing the user password for new
users. This is to fulfill backend expectations of password being
present while admin provides a one-time password to log into the
server and perform operations.

NOTE: this commit changes user.create and user.update responses
so that they are dictionaries rather than int.

(cherry picked from commit ffa3c61)
@anodos325 anodos325 merged commit 5ffc5e7 into stable/fangtooth Jan 28, 2025
1 check passed
@anodos325 anodos325 deleted the NAS-133758-25.04-RC.1 branch January 28, 2025 13:07
@bugclerk
Copy link
Contributor Author

JIRA ticket https://ixsystems.atlassian.net/browse/NAS-133758 is targeted to the following versions which have not received their corresponding PRs: 25.10

@bugclerk
Copy link
Contributor Author

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Jan 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants