fix(controller): correct client overwrites (#434)

oliverbaehler · web-flow · commit 125079676b89 · 2024-04-25T14:10:36.000+02:00
Signed-off-by: Oliver Bähler &lt;oliverbaehler@hotmail.com&gt;
diff --git a/main.go b/main.go
@@ -223,8 +223,10 @@ First match is used and can be specified multiple times as comma separated value
 
 	var clientOverride client.Reader
 
-	if !disableCaching {
+	if disableCaching {
 		clientOverride = mgr.GetAPIReader()
+	} else {
+		clientOverride = mgr.GetClient()
 	}
 
 	r, err = webserver.NewKubeFilter(listenerOpts, serverOpts, gates, rbReflector, clientOverride, mgr.GetClient())

Original file line number	Diff line number	Diff line change
`@@ -223,8 +223,10 @@ First match is used and can be specified multiple times as comma separated value`
`223`	`223`
`224`	`224`	`var clientOverride client.Reader`
`225`	`225`
`226`		`- if !disableCaching {`
	`226`	`+ if disableCaching {`
`227`	`227`	`clientOverride = mgr.GetAPIReader()`
	`228`	`+ } else {`
	`229`	`+ clientOverride = mgr.GetClient()`
`228`	`230`	`}`
`229`	`231`
`230`	`232`	`r, err = webserver.NewKubeFilter(listenerOpts, serverOpts, gates, rbReflector, clientOverride, mgr.GetClient())`