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.

Sign up for GitHub

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

Jump to bottom

[FEATURE REQUEST] allow password locking/unlocking in user.present on supported OS #62856

Closed
nicholasmhughes opened this issue Oct 10, 2022 · 0 comments · Fixed by #62857
Closed

[FEATURE REQUEST] allow password locking/unlocking in user.present on supported OS #62856

nicholasmhughes opened this issue Oct 10, 2022 · 0 comments · Fixed by #62857
Assignees
@nicholasmhughes
Labels
Feature new functionality including changes to functionality and code refactors, etc.

Comments

@nicholasmhughes
Copy link
Collaborator

Describe the solution you'd like
The user.present state currently allows declarative specification of user accounts, including passwords. However, it cannot currently declaratively determine the account to be in a locked/unlocked state. The shadow.lock_password execution module function (or appropriate unlock or OS-specific function) must be used outside of the user.present state in order to achieve the desired effect.

Describe alternatives you've considered
A module.run with onlyif/unless to check the state of the account.

Please Note
If this feature request would be considered a substantial change or addition, this should go through a SEP process here https://github.com/saltstack/salt-enhancement-proposals, instead of a feature request.
@nicholasmhughes nicholasmhughes added the Feature new functionality including changes to functionality and code refactors, etc. label Oct 10, 2022
@nicholasmhughes nicholasmhughes self-assigned this Oct 10, 2022
nicholasmhughes added a commit to nicholasmhughes/salt that referenced this issue Oct 10, 2022
@nicholasmhughes
fixes saltstack#62856 add password/account locking/unlocking in user.…
bb4b9a5 
…present state on supported operating systems
@nicholasmhughes nicholasmhughes mentioned this issue Oct 10, 2022
Merged
3 tasks
Ch3LL pushed a commit that referenced this issue Oct 11, 2022
@nicholasmhughes
fixes #62856 add password/account locking/unlocking in user.present s…
4e1bbec 
…tate on supported operating systems
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nicholasmhughes nicholasmhughes

Labels
Feature new functionality including changes to functionality and code refactors, etc.
Projects
None yet
Milestone
No milestone
1 participant
@nicholasmhughes