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

Allow nesting commands under minion names in master ACL config #208

Closed

Conversation

johnccfm
Copy link

The current code allows ACL's like:-

client_acl:
  # Allow thatch to execute anything.
  thatch:
    - .*

But it breaks horribly on ACL's like:-

client_acl:
  # Allow fred to use test and pkg, but only on "web*" minions.
  fred:
    - web*:
      - test.*
      - pkg.*

This patch fixes this - perhaps not in the safest or most elegant way, but it seems to work.

@johnccfm johnccfm closed this Mar 17, 2016
@gravyboat
Copy link
Contributor

Did you close this on purpose?

Edit: nevermind i realized you have #209.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants