saz / puppet-sudo

Manage sudo with Puppet on Debian-, RedHat- and SUSE-based linux distributions and some BSDs
Other
107 stars 215 forks source link

changed validate_single, rm 2 params, fixes #223 #309

Open tdlc opened 1 week ago

tdlc commented 1 week ago

To make validate_single to really only validate single files the implementation was changed. If validate_single is true visudo will be called with -f . In case validate_single is false all files will be validated. This makes sense because a single file could break the whole sudoers config, see issue #125. Before this commit all files would always be validated no matter which value validate_single had. This might be unwanted if an application installs some file with wrong rights 0440, see issue #223. Removed parameter delete_on_error because now an invalid file is never kept. When param was false it could also lead to infinite error messages in the invalid sudoers file. Removed parameter sudo_syntax_path as it is unused now. It cannot be used in puppet file's validate_cmd.