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.

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

sudo resource chokes on group strings #118

Closed
bby-bishopclark opened this issue Feb 7, 2018 · 1 comment
Closed

sudo resource chokes on group strings #118

bby-bishopclark opened this issue Feb 7, 2018 · 1 comment

Comments

@bby-bishopclark
Copy link

Cookbook version

3.5.3

Chef-client version

13.7.16

Platform Details

enterprise linux 7

Scenario:

https://github.com/chef-cookbooks/sudo/blob/master/providers/default.rb#L84 doesn't proper prepend group-marker to group list in string.

sudo 'app_admins'  do
  group 'web_admins,db_admins'
  commands  ['/sbin/service httpd restart']
end

-> /etc/sudoers.d/app_admins

%web_admins,db_admins ALL(ALL)=ALL /sbin/service httpd restart

Steps to Reproduce:

as in scenario

Expected Result:

%web_admins,%db_admins ALL(ALL)=ALL /sbin/service httpd restart

(with the % prepended on the second group)

Actual Result:

as in scenario - one leading % only

Hmm. It doesn't accept group=>['group1','group2'] right?

@tas50
Copy link
Contributor

tas50 commented Mar 13, 2018

This is also solved by 0375f15

@tas50 tas50 closed this as completed Mar 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants