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

ACL limit args #29153

Merged
Merged

Conversation

DmitryKuzmenko
Copy link
Contributor

This allows to limit function arguments values with 'external_auth' in master config.
The feature is implemented for #3077

@cachedout
Copy link
Contributor

Great! This might take a little time to review but I will try and get it in today or tomorrow.

@DmitryKuzmenko
Copy link
Contributor Author

@cachedout thank you!

@DmitryKuzmenko
Copy link
Contributor Author

I'll recheck tomorrow. The tests passed locally on my dev box.

@jfindlay jfindlay added Core relates to code central or existential to Salt Salt-API labels Nov 24, 2015
Dmitry Kuzmenko added 2 commits November 25, 2015 12:33
The method logic was changed in mock-1.1.0.
This mades the use of the method compatible with both <1.1.0 and >=1.1.0
cachedout pushed a commit that referenced this pull request Nov 25, 2015
@cachedout cachedout merged commit 89aee40 into saltstack:develop Nov 25, 2015
@cachedout
Copy link
Contributor

Looks really good! Merged.

@DmitryKuzmenko
Copy link
Contributor Author

@cachedout thank you!

@danlsgiga
Copy link
Contributor

This is a really awesome feature! Thanks a lot!

rallytime pushed a commit to rallytime/salt that referenced this pull request Jun 9, 2016
This adds documentation for the addition in saltstack#29153.
@thatsk
Copy link

thatsk commented May 21, 2018

can we get one example for package

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core relates to code central or existential to Salt Salt-API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants