Use correct log.Infof format for webhook sink audit output and fix var

naming

Signed-off-by: JoshVanL <[email protected]>

Author: JoshVanL

cmd/app/run.go

@@ -72,7 +72,7 @@ func buildRunCommand(stopCh <-chan struct{}, opts *options.Options) *cobra.Comma
 				return err
 			}
 
-			proxyOptions := &proxy.Config{
+			proxyConfig := &proxy.Config{
 				TokenReview:          opts.App.TokenPassthrough.Enabled,
 				DisableImpersonation: opts.App.DisableImpersonation,
 
@@ -85,7 +85,7 @@ func buildRunCommand(stopCh <-chan struct{}, opts *options.Options) *cobra.Comma
 
 			// Initialise proxy with OIDC token authenticator
 			p, err := proxy.New(restConfig, opts.OIDCAuthentication, opts.Audit,
-				tokenReviewer, secureServingInfo, proxyOptions)
+				tokenReviewer, secureServingInfo, proxyConfig)
 			if err != nil {
 				return err
 			}

pkg/proxy/handlers.go

@@ -181,7 +181,6 @@ func (p *Proxy) newErrorHandler() func(rw http.ResponseWriter, r *http.Request,
 
 		// Failed auth
 		case errUnauthorized:
-			klog.V(2).Infof("unauthenticated user request %s", r.RemoteAddr)
 			// If Unauthorized then error and report to audit
 			unauthedHandler.ServeHTTP(rw, r)
 			return

test/tools/audit-webhook/pkg/sink/sink.go

@@ -93,7 +93,7 @@ func (s *Sink) ServeHTTP(rw http.ResponseWriter, r *http.Request) {
 		http.Error(rw, err.Error(), http.StatusBadRequest)
 		return
 	}
-	log.Infof("%s: got events: %s", r.RemoteAddr, events)
+	log.Infof("%s: got events: %v", r.RemoteAddr, events)
 
 	s.mu.Lock()
 	defer s.mu.Unlock()