Change SuperAdminRoleName to SuperAdminStatus
SuperAdmin is not a role type, but it a status of a user that is tangentially related to a users role in an organization. This renames the variable to reflect that difference.pull/2249/head
parent
cbbf9d96d7
commit
6f66dd8a47
|
@ -33,11 +33,11 @@ func validRole(ctx context.Context) error {
|
|||
|
||||
// Chronograf User Roles
|
||||
const (
|
||||
MemberRoleName = "member"
|
||||
ViewerRoleName = "viewer"
|
||||
EditorRoleName = "editor"
|
||||
AdminRoleName = "admin"
|
||||
SuperAdminRoleName = "superadmin"
|
||||
MemberRoleName = "member"
|
||||
ViewerRoleName = "viewer"
|
||||
EditorRoleName = "editor"
|
||||
AdminRoleName = "admin"
|
||||
SuperAdminStatus = "superadmin"
|
||||
)
|
||||
|
||||
var (
|
||||
|
|
|
@ -197,7 +197,7 @@ func hasAuthorizedRole(u *chronograf.User, role string) bool {
|
|||
return true
|
||||
}
|
||||
}
|
||||
case roles.SuperAdminRoleName:
|
||||
case roles.SuperAdminStatus:
|
||||
// SuperAdmins should have been authorized before this.
|
||||
// This is only meant to restrict access for non-superadmins.
|
||||
return false
|
||||
|
|
|
@ -99,7 +99,7 @@ func NewMux(opts MuxOpts, service Service) http.Handler {
|
|||
return AuthorizedUser(
|
||||
service.Store,
|
||||
opts.UseAuth,
|
||||
roles.SuperAdminRoleName,
|
||||
roles.SuperAdminStatus,
|
||||
opts.Logger,
|
||||
next,
|
||||
)
|
||||
|
|
Loading…
Reference in New Issue