/juju

Add a Check for restricted methods #5727

Merged
merged 1 commit into from Jun 28, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@perrito666 @jujubot
@perrito666
Contributor

perrito666 commented Jun 28, 2016 •
Edited 1 time

Restricted root needed a check for restricted methods to avoid
having unprivileged users create models and call other unrestricted
root actions.
This is a temporary fix while a stronger authentication/acls solution is put in place.

(Review request: http://reviews.vapour.ws/r/5174/)
@perrito666
Add a Check for restricted methods 
Restricted root needed a check for restricted methods to avoid
having unprivileged users create models and call other unrestricted
root actions
a5f567a
@perrito666
Contributor

perrito666 commented Jun 28, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Jun 28, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju
@jujubot
Contributor

jujubot commented Jun 28, 2016

Build failed: Does not match ['fixes-1596493', 'fixes-1595686', 'fixes-1586890']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8295
@perrito666
Contributor

perrito666 commented Jun 28, 2016

$$JFDI$$ Approved by Ian
@jujubot
Contributor

jujubot commented Jun 28, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit 70f7169 into juju:master Jun 28, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment