From fd14a5794ef65b58449347382c386f2db9230831 Mon Sep 17 00:00:00 2001 From: "Benjamin A. Petersen" Date: Mon, 12 Feb 2024 12:38:57 -0500 Subject: [PATCH] ldap upstream watcher: rename local var for clarity --- .../ldapupstreamwatcher/ldap_upstream_watcher.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/internal/controller/supervisorconfig/ldapupstreamwatcher/ldap_upstream_watcher.go b/internal/controller/supervisorconfig/ldapupstreamwatcher/ldap_upstream_watcher.go index e5b2bc7a2..90a910d99 100644 --- a/internal/controller/supervisorconfig/ldapupstreamwatcher/ldap_upstream_watcher.go +++ b/internal/controller/supervisorconfig/ldapupstreamwatcher/ldap_upstream_watcher.go @@ -209,9 +209,9 @@ func (c *ldapWatcherController) Sync(ctx controllerlib.Context) error { requeue := false validatedUpstreams := make([]upstreamprovider.UpstreamLDAPIdentityProviderI, 0, len(actualUpstreams)) for _, upstream := range actualUpstreams { - valid, requestedRequeue := c.validateUpstream(ctx.Context, upstream) - if valid != nil { - validatedUpstreams = append(validatedUpstreams, valid) + validProvider, requestedRequeue := c.validateUpstream(ctx.Context, upstream) + if validProvider != nil { + validatedUpstreams = append(validatedUpstreams, validProvider) } if requestedRequeue { requeue = true