-
Notifications
You must be signed in to change notification settings - Fork 18
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
user-profile: add authorised-users field #273
Comments
Second thoughts after a chat with @datamel who pointed out the authorised users list isn't necessarily the most natural choice for this configuration.
Here's a revised proposal for a new configuration that configures both the hubs and users in one go: <config-name> = {
('hub-one', 'https://hubone.com/cylc/'): [
'user-a',
'user-b',
],
('hub-two', 'https://two.hub/'): [
'user-c',
]
}
This should also provide a solution for #238 and #239. Some of this should be set by the site, some of it should be set by the user. Needs a little more thought. |
Yes thanks @oliver-sanders , sending authorized users across to the UI is not going to provide an accurate drop down box for the ui, since these are users who are able to access this ui-server (not users who the user is permitted to connect to). With regards to the config suggested above, what if this config, listing possible combinations was set by site, and users just configured usernames they wish to access, the realms could then be auto-filled as a drop down in the ui? |
It might make sense for the site to configure some of the users a well as the hubs. For example: <config-name> = {
('operations', 'https://cylc.operations'): [
'operations-user'
]
} |
Discussed in VC on 18/11/21. This should be configurable, by username. Pass this configurable list of usernames to the ui, to populate the drop down menu for users to access another ui-server. |
This is trickier than anticipated, bumping to 1.x. |
Add a new field called "authorized users" (or similar) to the user-profile containing all users listed in the user's authentication config when running in multi-user mode.
This field will be used by the Cylc UI to allow quick access to the listed UI Servers via a user-name dropdown/filter box in the top-left of the UI.
Example:
In multi-user mode (when the application class is
CylcHubApp
):In single user mode:
Note, we don't need to check that the listed users are authorised, if they are not that will become obvious when the user tries to access their UIS.
Pull requests welcome!
The text was updated successfully, but these errors were encountered: