use match

odelcroi · sandhose · odelcroi · commit 10bda44f666e · 2025-07-17T14:37:28.000+02:00
Co-authored-by: Quentin Gliech &lt;quenting@element.io&gt;
diff --git a/crates/config/src/sections/upstream_oauth2.rs b/crates/config/src/sections/upstream_oauth2.rs
@@ -118,8 +118,10 @@ impl ConfigurationSection for UpstreamOAuth2Config {
                 }
             }
 
-            if !provider.claims_imports.localpart.on_conflict.is_default()
-                && !matches!(
+            if matches!(
+                provider.claims_imports.localpart.on_conflict,
+                OnConflict::Add
+            ) && !matches!(
                     provider.claims_imports.localpart.action,
                     ImportAction::Force | ImportAction::Require
                 )

Original file line number	Diff line number	Diff line change
`@@ -118,8 +118,10 @@ impl ConfigurationSection for UpstreamOAuth2Config {`
`118`	`118`	`}`
`119`	`119`	`}`
`120`	`120`
`121`		`- if !provider.claims_imports.localpart.on_conflict.is_default()`
`122`		`- && !matches!(`
	`121`	`+ if matches!(`
	`122`	`+ provider.claims_imports.localpart.on_conflict,`
	`123`	`+ OnConflict::Add`
	`124`	`+ ) && !matches!(`
`123`	`125`	`provider.claims_imports.localpart.action,`
`124`	`126`	`ImportAction::Force \| ImportAction::Require`
`125`	`127`	`)`