search

ahuffman / ansible-sudoers

Controls the configuration of the default /etc/sudoers file and included files/directories.
MIT License
108 stars 45 forks source link

Misc fixes #22

Closed dev-zero closed 6 years ago

dev-zero commented 6 years ago

The only notable thing is the mode on /etc/sudoers.d which on most distros I've checked is not 0755 but 0750. Quoting the validate command is based on the example given in the Ansible documentation. And using map('basename') instead of a regex is simply easier to read and understand.

wtcross commented 6 years ago

Thanks for your contribution! This looks great.

ahuffman commented 6 years ago

Thanks for meeting that in, fell off my radar.

On Mar 13, 2018 11:31 AM, "Tyler Cross" notifications@github.com wrote:

Thanks for your contribution! This looks great.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/wtcross/ansible-sudoers/pull/22#issuecomment-372707292, or mute the thread https://github.com/notifications/unsubscribe-auth/AEI6LYREqv0x8qh0HDUS3JxU65D8sFNNks5td-ZZgaJpZM4RyU6q .