Commitb76f373

committed

Add some comments

1 parentdb04d67 commitb76f373Copy full SHA for b76f373

File tree

-5

lines changed

-5

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@ func Authorize(logger slog.Logger, auth *rbac.RegoAuthorizer, action rbac.Action`
`20`	`20`	`returnfunc(next http.Handler) http.Handler {`
`21`	`21`	`returnhttp.HandlerFunc(func(rw http.ResponseWriter,r*http.Request) {`
`22`	`22`	`roles:=UserRoles(r)`
`23`		`-object:=authObject(r)`
	`23`	`+object:=rbacObject(r)`
`24`	`24`
`25`	`25`	`ifobject.Type=="" {`
`26`	`26`	`panic("developer error: auth object has no type")`
`@@ -72,7 +72,7 @@ func Authorize(logger slog.Logger, auth *rbac.RegoAuthorizer, action rbac.Action`
`72`	`72`	`typeauthObjectKeystruct{}`
`73`	`73`
`74`	`74`	`// APIKey returns the API key from the ExtractAPIKey handler.`
`75`		`-funcauthObject(r*http.Request) rbac.Object {`
	`75`	`+funcrbacObject(r*http.Request) rbac.Object {`
`76`	`76`	`obj,ok:=r.Context().Value(authObjectKey{}).(rbac.Object)`
`77`	`77`	`if!ok {`
`78`	`78`	`panic("developer error: auth object middleware not provided")`

Lines changed: 7 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -146,9 +146,11 @@ func IsOrgRole(roleName string) (string, bool) {`
`146`	`146`	`}`
`147`	`147`
`148`	`148`	`// OrganizationRoles lists all roles that can be applied to an organization user`
`149`		`-// in the given organization.`
	`149`	`+// in the given organization. This is the list of available roles,`
	`150`	`+// and specific to an organization.`
	`151`	`+//`
`150`	`152`	`// This should be a list in a database, but until then we build`
`151`		`-//the list from the builtins.`
	`153`	`+// the list from the builtins.`
`152`	`154`	`funcOrganizationRoles(organizationID uuid.UUID) []string {`
`153`	`155`	`varroles []string`
`154`	`156`	`for_,roleF:=rangebuiltInRoles {`
`@@ -166,8 +168,10 @@ func OrganizationRoles(organizationID uuid.UUID) []string {`
`166`	`168`	`}`
`167`	`169`
`168`	`170`	`// SiteRoles lists all roles that can be applied to a user.`
	`171`	`+// This is the list of available roles, and not specific to a user`
	`172`	`+//`
`169`	`173`	`// This should be a list in a database, but until then we build`
`170`		`-//the list from the builtins.`
	`174`	`+// the list from the builtins.`
`171`	`175`	`funcSiteRoles() []string {`
`172`	`176`	`varroles []string`
`173`	`177`	`forrole:=rangebuiltInRoles {`

Comments

(0)