Fix panic on serve logging and add message (#98)

* Fix panic on serve logging and add message * Also do the same on errors
Netflix · Oct 15, 2021 · 43ee64d · 43ee64d
1 parent b7a0ff5
commit 43ee64d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/pkg/httpAuth/mtls/certificate.go b/pkg/httpAuth/mtls/certificate.go
@@ -110,7 +110,6 @@ func (wc *wrappedCertificate) autoRefresh() {
 			select {
 			case event, ok := <-watcher.Events:
 				if !ok {
-					logging.Log.Warn("problem with mTLS file watcher")
 					return
 				}
 				logging.Log.Debugf("event received: %v", event)
@@ -119,10 +118,9 @@ func (wc *wrappedCertificate) autoRefresh() {
 				}
 			case watcherError, ok := <-watcher.Errors:
 				if !ok {
-					logging.Log.Warn("problem with mTLS file watcher")
 					return
 				}
-				logging.Log.Error(watcherError)
+				logging.LogError(watcherError, "problem with mTLS file watcher")
 			}
 		}
 	}()

diff --git a/pkg/server/middleware.go b/pkg/server/middleware.go
@@ -92,7 +92,7 @@ func AWSHeaderMiddleware(next http.HandlerFunc) http.HandlerFunc {
 			"user-agent":       ua,
 			"path":             r.URL.Path,
 			"metadata_version": metadataVersion,
-		}).Info()
+		}).Info("Running AWS Header Middleware")
 		next.ServeHTTP(w, r)
 	}
 }