-
Notifications
You must be signed in to change notification settings - Fork 52
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
Fix for Issue 179 #180
Fix for Issue 179 #180
Conversation
@@ -25,10 +25,12 @@ end}. | |||
%% @doc If riak control's authentication mode (riak_control.auth.mode) | |||
%% is set to 'userlist' then this is the list of usernames and | |||
%% passwords for access to the admin panel. | |||
%% To create users with given names, add entries of the format: | |||
%% riak_control.auth.user.«user name».password = «password» |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd prefer not to use guillemot in the schema. We're currently having utf8 vs. latin1 issues anyway, surely those characters are > 127 in latin1 (if they even exist in it).
Eng +1 |
CSE +1 |
Prod +1 |
👍, however please open a PR for resulting test failure which will occur here: https://github.com/basho/riak_test/blob/master/tests/riak_control_authentication.erl |
@cmeiklejohn I don't see how that will fail, the configs in that test are unchanged. |
Ah, yes. You are right. Merge away. |
Added to 2.0-RC milestone. |
This fix is designed to address issue #179 by changing the default username from "user" to "admin" and creating some additional documentation to explain how to add additional users.