Skip to content

Commit

Permalink
fix: saml logoutall - retain logoutAllSupported flag when resetting…
Browse files Browse the repository at this point in the history
… an auth config back to unused after it was disabled.
  • Loading branch information
andreas-kupries committed Oct 10, 2024
1 parent bf39f57 commit 38a7138
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion pkg/controllers/management/auth/auth_config.go
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,13 @@ func (ac *authConfigController) refreshUsers(obj *v3.AuthConfig) error {

// resetAuthConfig takes an Auth Config as a map and deletes all entries except those with basic metadata fields.
func resetAuthConfig(cfg map[string]any) {
retainFields := map[string]bool{"apiVersion": true, "kind": true, "metadata": true, "type": true}
retainFields := map[string]bool{
"apiVersion": true,
"kind": true,
"metadata": true,
"type": true,
"logoutAllSupported": true,
}
for field := range cfg {
if !retainFields[field] {
delete(cfg, field)
Expand Down

0 comments on commit 38a7138

Please sign in to comment.