Bring RHEL secure_path settings in line with other OSes, supports PE binary location

saz / puppet-sudo

Manage sudo with Puppet on Debian-, RedHat- and SUSE-based linux distributions and some BSDs

Other

105 stars 215 forks source link

Bring RHEL secure_path settings in line with other OSes, supports PE binary location #122

Closed rnelson0 closed 9 years ago

rnelson0 commented 9 years ago

Without this patch you cannot run Puppet Enterprise without pathing (sudo /usr/local/bin/puppet) but also have issues with r10k as it looks in $PATH for the puppet binary. This removes the need to sudo su - just to run r10k.

ffrank commented 9 years ago

Can this not have security implications for some users?

rnelson0 commented 9 years ago

It could, but it's already part of the Ubuntu and Wheezy settings and not set at all in the other files. An alternative might be to change the file source to a template and only add that when is_pe is true, but since the PR is just bringing these OSes into line with the other OS files I do not think that is needed