Merge pull request #3110 from gravitl/hotfix/release-v0.25.0/rac-auto-disable

fix: user platform role check in rac auto disable
This commit is contained in:
Abhishek K 2024-09-09 06:42:11 +04:00 committed by GitHub
commit e3c3a27f80
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -47,7 +47,10 @@ func racAutoDisableHook() error {
continue
}
for _, client := range clients {
if (client.OwnerID == user.UserName) && !user.IsAdmin && !user.IsSuperAdmin && client.Enabled {
if (client.OwnerID == user.UserName) &&
user.PlatformRoleID != models.SuperAdminRole &&
user.PlatformRoleID != models.AdminRole &&
client.Enabled {
slog.Info(fmt.Sprintf("disabling ext client %s for user %s due to RAC autodisabling", client.ClientID, client.OwnerID))
if err := disableExtClient(&client); err != nil {
slog.Error("error disabling ext client in RAC autodisable hook", "error", err)