Cockroach: ui: add user/login system

Created on 26 Apr 2016  路  8Comments  路  Source: cockroachdb/cockroach

Add a login system to the UI. The backend has a concept of user, but it isn't currently exposed in the UI. We need a "users" page to list the existing users, add/remove users and change privileges. Users would have to login to the UI and, depending on their privileges, would have a restricted view of the system. This work should be done in TBD stages.

A-webui-security C-enhancement

Most helpful comment

One final fix here on the mocks is to remove the side nav bar on the login page since those buttons won't work anyway. cc @nstewart

All 8 comments

Updating this GitHub issue with finalized 1.1 scope - just basic user login with error messages and login ID in the top right. We also need to consider how these logins reflect in logging in case users want to audit usage. cc @kuanluo

https://github.com/cockroachdb/cockroach/blob/69f24571ef0bdc75fd78da6cb400c0933e586c8e/pkg/security/auth.go#L126-L143 --> sampling of existing error messages.

Login with field error:
image

If the error is more generic:
image

Log out button with ID top right:
image

cc: @bdarnell @mrtracy

One final fix here on the mocks is to remove the side nav bar on the login page since those buttons won't work anyway. cc @nstewart

punting to 1.2 due to risk of this breaking user workflow without sufficient QA time.

If the entire UI requires login, and login requires the database to be working properly, will we no longer be able to use the UI (including the debug pages) to debug broken clusters?

cc @piyush-singh re: @a-robinson 's question. Let's make sure we don't remove an important debugging workflow

Closing this since login is implemented and now enabled by default in secure clusters. #29270 tracks @a-robinson's concern that this will block debugging flows.

Was this page helpful?
0 / 5 - 0 ratings