I know it's halloween, but still... we should rewrite this in Ruby

Signed-off-by: Alex Snaps <[email protected]>
Kuadrant · Oct 24, 2024 · fdc7c7b · fdc7c7b
1 parent b445436
commit fdc7c7b
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/pkg/evaluators/identity.go b/pkg/evaluators/identity.go
@@ -199,11 +199,13 @@ func (config *IdentityConfig) ResolveExtendedProperties(pipeline auth.AuthPipeli
 	authJSON := pipeline.GetAuthorizationJSON()
 
 	for _, extendedProperty := range config.ExtendedProperties {
-		resolved, err := extendedProperty.ResolveFor(extendedIdentityObject, authJSON)
-		if err != nil {
-			return nil, err
+		if extendedProperty.Value != nil {
+			resolved, err := extendedProperty.ResolveFor(extendedIdentityObject, authJSON)
+			if err != nil {
+				return nil, err
+			}
+			extendedIdentityObject[extendedProperty.Name] = resolved
 		}
-		extendedIdentityObject[extendedProperty.Name] = resolved
 	}
 
 	return extendedIdentityObject, nil