fix: suppress deprecation warning for userAttr when not set (#4539)

nabokihms · web-flow · commit 489e37d07ad9 · 2026-02-13T13:19:20.000+01:00
Signed-off-by: maksim.nabokikh &lt;max.nabokih@gmail.com&gt;
diff --git a/connector/ldap/ldap.go b/connector/ldap/ldap.go
@@ -198,7 +198,9 @@ func userMatchers(c *Config, logger *slog.Logger) []UserMatcher {
 		return c.GroupSearch.UserMatchers
 	}
 
-	logger.Warn(`use "groupSearch.userMatchers" option instead of "userAttr/groupAttr" fields`, "deprecated", true)
+	if c.GroupSearch.UserAttr != "" || c.GroupSearch.GroupAttr != "" {
+		logger.Warn(`use "groupSearch.userMatchers" option instead of "userAttr/groupAttr" fields`, "deprecated", true)
+	}
 	return []UserMatcher{
 		{
 			UserAttr:           c.GroupSearch.UserAttr,

Original file line number	Diff line number	Diff line change
`@@ -198,7 +198,9 @@ func userMatchers(c Config, logger slog.Logger) []UserMatcher {`
`198`	`198`	`return c.GroupSearch.UserMatchers`
`199`	`199`	`}`
`200`	`200`
`201`		- logger.Warn(`use "groupSearch.userMatchers" option instead of "userAttr/groupAttr" fields`, "deprecated", true)
	`201`	`+ if c.GroupSearch.UserAttr != "" \|\| c.GroupSearch.GroupAttr != "" {`
	`202`	+ logger.Warn(`use "groupSearch.userMatchers" option instead of "userAttr/groupAttr" fields`, "deprecated", true)
	`203`	`+ }`
`202`	`204`	`return []UserMatcher{`
`203`	`205`	`{`
`204`	`206`	`UserAttr: c.GroupSearch.UserAttr,`