- Notifications
You must be signed in to change notification settings - Fork1.1k
feat: remove "view all users" from members#8447
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
If omitted, the caller does not have permission to view said data
kylecarbs commentedJul 13, 2023
@Emyrk I think we should allow users to see who is creating and maintaining templates. It's a worse experience for the user to not see that... they'll have no action to take if something is broken. |
Emyrk commentedJul 14, 2023
I paused this PR because there is a lot of cases like that. There is a call today about it and a thread in slack + a notion page. Essentially this is trickier than anticipated, so I broke the functional components of this PR up into different PRs and will likely close this. |
Uh oh!
There was an error while loading.Please reload this page.
Moved to#8650
What this does
Removes the rbac permission for users to read all other users on the site.
Closes#5002
Changes
Permission Changes
ResourceTypeUsernow owned byuser_idof itself.membercouldreadall Users -->membercan read only themselvesOpen issues to resolve
All templates fetch the creator name. This is no longer able to be read.
coder/coderd/templates.go
Lines 682 to 692 in04a2cae
created_byandcreator_usernameis now omitted if the user cannot view the creator. Meaning they can see templates created by themselves. Admins can still view all users.User counts are still being read
GetUserswith sql filter rather than filtering the returned list. Now all counts are counts of the readable users for any given caller.Group members still return all users
If you grant a member
adminon a template, they can see the permissions page. This page has a list of users/groups that can use/admin the template. Should this page show users they cannot read?