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

Add "is root" attribute to users #1623

Closed
3 tasks done
stopfstedt opened this issue Oct 11, 2016 · 4 comments
Closed
3 tasks done

Add "is root" attribute to users #1623

stopfstedt opened this issue Oct 11, 2016 · 4 comments
Assignees

Comments

@stopfstedt
Copy link
Member

stopfstedt commented Oct 11, 2016

  • root users can access everything at all times, instance-wide.
  • check for root needs to be wired into all voters, the root-check will need to precede any other authorization checks within these voters; essentially short-circuiting those.
  • exclusive capabilities of root users:
    • can create schools
    • can delete schools
    • can flag other users as "root"
  • add 'isRoot' attribute to User entity
  • create console commands for setting/un-setting the isRoot attribute on any given user
  • ensure that only root users are able to modify the isRoot attribute on any given user via the API
@stopfstedt
Copy link
Member Author

refs #1511

@stopfstedt stopfstedt self-assigned this Oct 11, 2016
@stopfstedt
Copy link
Member Author

make sure to bake an extra perms check into the user controller's POST and PUT action to ensure that the current user has rights to set/modify the 'root' flag.

@jrjohnson
Copy link
Member

@stopfstedt - I think this can be closed, yes?

@stopfstedt
Copy link
Member Author

the answer is YES.

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

No branches or pull requests

2 participants