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

sql: add root user to system.users table during bootstrap and add authentication #9877

Closed
asubiotto opened this issue Oct 11, 2016 · 3 comments

Comments

@asubiotto
Copy link
Contributor

asubiotto commented Oct 11, 2016

For authentication, the root user must exist in the system.users table. A password should also be chosen at this stage. #9794 adds a backdoor for clients authenticating as security.RootUser. This should be removed with this change.

@asubiotto asubiotto changed the title sql: Add root user to system.users table during bootstrap sql: add root user to system.users table during bootstrap and add authentication Oct 11, 2016
@asubiotto
Copy link
Contributor Author

This should happen after #9404.

@bdarnell
Copy link
Contributor

As I said in #9794, I would prefer if it were impossible to give the root user a password and it used certificate-only authentication. Then the bootstrapping could be non-interactive. (I think it would be nice if root existed in system.users, but I'd also be OK with a special-case that would allow root to be used regardless of the state of that table).

@asubiotto
Copy link
Contributor Author

Closing issue because root will have cert only authentication. Since we never have to check for the existence of root, there doesn't seem to be a need to add root to system.users

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