-
Notifications
You must be signed in to change notification settings - Fork 22
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
(#145) Added the ability to control pwhistory.conf #146
Conversation
README.md
Outdated
pam::faillock_admin_group: 'wheel' | ||
``` | ||
|
||
### Managing /etc/security/pwhistory.conf | ||
|
||
To manage pwhistory with ``/etc/security/pwhistory.conf`` instead of inline parameters in the auth files set the following in hieradata: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd drop the text instead of inline parameters in the auth files
.
Also shouldn't those be single backticks, not double? (Double backticks render as single backticks, IIRC, not as a monospace font block.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The double backticks are all over the place in the readme and they appear to render without issues. That said, I'll go through and try to be consistent with the number of backticks being used.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm guessing the .err
files were added unintentionally?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, absolutely
Fixes #145