log ldap warnings instead of returning them to end user

hashiblaum · hashiblaum · commit cf80b6d1844a · 2024-12-09T16:13:32.000-05:00
diff --git a/builtin/credential/ldap/backend.go b/builtin/credential/ldap/backend.go
@@ -125,10 +125,9 @@ func (b *backend) Login(ctx context.Context, req *logical.Request, username stri
 	}
 
 	for _, warning := range c.Warnings {
-		if b.Logger().IsDebug() {
-			b.Logger().Debug(string(warning))
+		if b.Logger().IsWarn() {
+			b.Logger().Warn(string(warning))
 		}
-		ldapResponse.AddWarning(string(warning))
 	}
 
 	var allGroups []string

Original file line number	Diff line number	Diff line change
`@@ -125,10 +125,9 @@ func (b backend) Login(ctx context.Context, req logical.Request, username stri`
`125`	`125`	`}`
`126`	`126`
`127`	`127`	`for _, warning := range c.Warnings {`
`128`		`- if b.Logger().IsDebug() {`
`129`		`- b.Logger().Debug(string(warning))`
	`128`	`+ if b.Logger().IsWarn() {`
	`129`	`+ b.Logger().Warn(string(warning))`
`130`	`130`	`}`
`131`		`- ldapResponse.AddWarning(string(warning))`
`132`	`131`	`}`
`133`	`132`
`134`	`133`	`var allGroups []string`