Skip to content

feat: Implied 'member' roles for site and organization #1917

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.

Sign up for GitHub

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

Merged
merged 9 commits into from
Jun 1, 2022
Merged

feat: Implied 'member' roles for site and organization #1917

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
501e581
feat: Member roles are implied and never exlpicitly added
Emyrk May 31, 2022
51521ca
Rename "GetAllUserRoles" to "GetAuthorizationRoles"
Emyrk May 31, 2022
844c9ce
feat: Add migration to remove implied roles
Emyrk May 31, 2022
28e3957
Add comment to sql
Emyrk May 31, 2022
058e7ab
test: Update test
Emyrk May 31, 2022
9a3d38e
handle null string array
Emyrk May 31, 2022
5060443
Merge remote-tracking branch 'origin/main' into stevenmasley/implied_…
Emyrk May 31, 2022
6a89023
rename user auth role middleware
Emyrk Jun 1, 2022
ccecdda
fixup! rename user auth role middleware
Emyrk Jun 1, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fixup! rename user auth role middleware
  • Loading branch information
@Emyrk
Emyrk committed Jun 1, 2022
commit ccecddad95cf7682c6089637ea53fff4a733ea80
2 changes: 1 addition & 1 deletion coderd/users.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -473,7 +473,7 @@ func (api *API) userRoles(rw http.ResponseWriter, r *http.Request) {
func (api *API) putUserRoles(rw http.ResponseWriter, r *http.Request) {
// User is the user to modify.
user := httpmw.UserParam(r)
roles := httpmw.UserAuthorizationRoles(r)
roles := httpmw.AuthorizationUserRoles(r)
apiKey := httpmw.APIKey(r)

if apiKey.UserID == user.ID {
Expand Down