/juju

Made RemoveUsers also revoke permissions. #6074

Merged
merged 1 commit into from Aug 25, 2016

Conversation

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

perrito666 commented Aug 23, 2016 •
Edited 1 time

When a users is "deleted" the permissions that user has
need to be removed too.

(Review request: http://reviews.vapour.ws/r/5517/)
@perrito666
Contributor

perrito666 commented Aug 24, 2016

$$fixes-1614161$$
@jujubot
Contributor

jujubot commented Aug 24, 2016

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

jujubot commented Aug 25, 2016

Build failed: Does not match ['fixes-1616584', 'fixes-1611514', 'fixes-1475212']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8926
@anastasiamac
Member

anastasiamac commented Aug 25, 2016

$$JFDI$$
@jujubot
Contributor

jujubot commented Aug 25, 2016

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

jujubot commented Aug 25, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8928
@perrito666
Made RemoveUsers also revoke permissions. 
When a users is "deleted" the permissions that user has
need to be unaccessible too, so UserAccess now checks
for local user existence.
75d3a08
@perrito666
Contributor

perrito666 commented Aug 25, 2016

$$JFDI$$
@jujubot
Contributor

jujubot commented Aug 25, 2016

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

@jujubot jujubot merged commit 930704b into juju:master Aug 25, 2016

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