AccessControl: Use RBACBuiltInRoleAssignmentEnabled in the frontend instead of FeatureToggle (#49261)

* AccessControl: Frontend changes to basic role enablement
This commit is contained in:
Gabriel MABILLE 2022-05-20 08:53:28 +02:00 committed by GitHub
parent 59c8c0d56b
commit 689f51599a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
6 changed files with 7 additions and 4 deletions

View File

@ -163,6 +163,7 @@ export interface GrafanaConfig {
verifyEmailEnabled: boolean;
oauth: OAuthSettings;
rbacEnabled: boolean;
rbacBuiltInRoleAssignmentEnabled: boolean;
disableUserSignUp: boolean;
loginHint: string;
passwordHint: string;

View File

@ -58,6 +58,7 @@ export class GrafanaBootConfig implements GrafanaConfig {
verifyEmailEnabled = false;
oauth: OAuthSettings = {};
rbacEnabled = true;
rbacBuiltInRoleAssignmentEnabled = false;
disableUserSignUp = false;
loginHint = '';
passwordHint = '';

View File

@ -106,6 +106,7 @@ func (hs *HTTPServer) getFrontendSettingsMap(c *models.ReqContext) (map[string]i
"verifyEmailEnabled": setting.VerifyEmailEnabled,
"sigV4AuthEnabled": setting.SigV4AuthEnabled,
"rbacEnabled": hs.Cfg.RBACEnabled,
"rbacBuiltInRoleAssignmentEnabled": hs.Cfg.RBACBuiltInRoleAssignmentEnabled,
"exploreEnabled": setting.ExploreEnabled,
"helpEnabled": setting.HelpEnabled,
"profileEnabled": setting.ProfileEnabled,

View File

@ -112,8 +112,8 @@ export class ContextSrv {
return config.rbacEnabled;
}
accessControlBuiltinRefactorEnabled(): boolean {
return Boolean(config.featureToggles['accesscontrol-builtins']);
accessControlBuiltInRoleAssignmentEnabled(): boolean {
return config.rbacBuiltInRoleAssignmentEnabled;
}
licensedAccessControlEnabled(): boolean {

View File

@ -32,7 +32,7 @@ export function fetchACOptions(): ThunkResult<void> {
dispatch(acOptionsLoaded(options));
}
if (
!contextSrv.accessControlBuiltinRefactorEnabled() &&
contextSrv.accessControlBuiltInRoleAssignmentEnabled() &&
contextSrv.licensedAccessControlEnabled() &&
contextSrv.hasPermission(AccessControlAction.ActionBuiltinRolesList)
) {

View File

@ -31,7 +31,7 @@ const UsersTable: FC<Props> = (props) => {
}
if (
!contextSrv.accessControlBuiltinRefactorEnabled() &&
contextSrv.accessControlBuiltInRoleAssignmentEnabled() &&
contextSrv.hasPermission(AccessControlAction.ActionBuiltinRolesList)
) {
const builtInRoles = await fetchBuiltinRoles(orgId);