fix: don't update attribute mapping if nil (supabase#1665)

LashaJini · Nov 13, 2024 · d121d83 · d121d83
1 parent 7275c8d
commit d121d83
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/internal/api/ssoadmin.go b/internal/api/ssoadmin.go
@@ -349,10 +349,13 @@ func (a *API) adminSSOProvidersUpdate(w http.ResponseWriter, r *http.Request) er
 		}
 	}
 
-	updateAttributeMapping := !provider.SAMLProvider.AttributeMapping.Equal(&params.AttributeMapping)
-	if updateAttributeMapping {
-		modified = true
-		provider.SAMLProvider.AttributeMapping = params.AttributeMapping
+	updateAttributeMapping := false
+	if params.AttributeMapping.Keys != nil {
+		updateAttributeMapping = !provider.SAMLProvider.AttributeMapping.Equal(&params.AttributeMapping)
+		if updateAttributeMapping {
+			modified = true
+			provider.SAMLProvider.AttributeMapping = params.AttributeMapping
+		}
 	}
 
 	nameIDFormat := ""