Skip to content

Commit

Permalink
Merge pull request #72 from kyma-project/fix/cf-admin2
Browse files Browse the repository at this point in the history
fix sap.ids prefix
  • Loading branch information
szeort authored Sep 30, 2024
2 parents acdddf7 + 3a07e18 commit 750d87d
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions controllers/cfapi_auth.go
Original file line number Diff line number Diff line change
Expand Up @@ -69,11 +69,13 @@ func (r *CFAPIReconciler) assignCfAdministrators(ctx context.Context, subjects [
}
}

oids_subjects := make([]rbacv1.Subject, len(_subjects))
//add prefix sap.ids: for all user names without prefix
for _, subject := range _subjects {
for i, subject := range _subjects {
if subject.Kind == "User" && !strings.HasPrefix(subject.Name, OIDC_USER_PREFIX) {
subject.Name = OIDC_USER_PREFIX + subject.Name
}
oids_subjects[i] = subject
}

rb := &rbacv1.RoleBinding{
Expand All @@ -89,7 +91,7 @@ func (r *CFAPIReconciler) assignCfAdministrators(ctx context.Context, subjects [
Kind: "ClusterRole",
Name: "korifi-controllers-admin",
},
Subjects: _subjects,
Subjects: oids_subjects,
}

userNames := make([]string, len(_subjects))
Expand Down

0 comments on commit 750d87d

Please sign in to comment.