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.

Sign up for GitHub

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

Jump to bottom

Remove permissions config #2199

Closed
mberhault opened this issue Aug 21, 2015 · 0 comments
Closed

Remove permissions config #2199

mberhault opened this issue Aug 21, 2015 · 0 comments
Assignees
@mberhault
Milestone
Beta

Comments

@mberhault
Copy link
Contributor

Follow-up to #2089 (lock down kv endpoints) and needed for #2090 (make configs aware of structured data layout).

Permissions are no longer adjustable on the KV endpoint: system users only (root and node).
@mberhault mberhault self-assigned this Aug 21, 2015
@tamird tamird modified the milestone: Beta Aug 21, 2015
@mberhault mberhault mentioned this issue Aug 22, 2015
Merged
mberhault pushed a commit that referenced this issue Aug 23, 2015
Remove permission config.
b3de776 
Fixes #2199

Mostly code removal. Some tests (eg: storage/replicate_test.go) used
the permission config to test gossip. Changed to use other configs.

I have kept the Read|Write flags on requests, they may still come in
handy.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mberhault mberhault

Labels
None yet
Projects
None yet
Milestone
Beta
Development

No branches or pull requests
2 participants
@tamird @mberhault