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

Remove admin_secret from riak-cs.conf and stanchion.conf [JIRA: RCS-307] #1274

Closed
kuenishi opened this issue Dec 7, 2015 · 8 comments
Closed
Assignees
Milestone

Comments

@kuenishi
Copy link
Contributor

kuenishi commented Dec 7, 2015

No description provided.

@kuenishi kuenishi added this to the 2.1.1 milestone Dec 7, 2015
@Basho-JIRA Basho-JIRA changed the title Remove admin_secret from riak-cs.conf Remove admin_secret from riak-cs.conf [JIRA: RCS-307] Dec 7, 2015
@shino
Copy link
Contributor

shino commented Dec 7, 2015

Maybe, stanchion too.

@kuenishi kuenishi changed the title Remove admin_secret from riak-cs.conf [JIRA: RCS-307] Remove admin_secret from riak-cs.conf and stanchion.conf [JIRA: RCS-307] Dec 7, 2015
@kuenishi
Copy link
Contributor Author

kuenishi commented Dec 7, 2015

@kuenishi
Copy link
Contributor Author

@shino
Copy link
Contributor

shino commented Dec 14, 2015

Memo: doc change needed.

@kuenishi
Copy link
Contributor Author

Change summary:

Riak CS and Stanchion both used riak-cs.conf or stanchion.conf to tell the system who was the administrator, with admin.key and admin.secret. Writing down admin.secret in non-encrypted form may risk the whole system of the secret being stolen. Administrator is able to (1) list all users, (2) disable/enable other users, (3) changing user accounts. Workaround for this is probably encrypting the partition that includes /etc/riak-cs directory.

These changes will enable Riak CS and Stanchion start up without admin.secret specified in configuration files, while admin.key is still needed. Even if admin.secret is written in those files, Riak CS and Stanchion ignores them. Initial procedure to create the very first account starting anonymous_user_creation = true does not change, though.

@shino
Copy link
Contributor

shino commented Dec 18, 2015

Another related PR: basho/riak_cs_multibag#33

@TJC
Copy link

TJC commented Nov 22, 2016

hey guys -- could you update http://docs.basho.com/riak/cs/2.1.1/cookbooks/configuration/riak-cs/ to reflect the current correct and best practices in regard to the admin secret? I'm confused.

@ParthKolekar
Copy link

Plus one. Do we need to put the admin secrets in the file or not?

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

No branches or pull requests

5 participants