addressed more pr review comments

pkg/http-server/start.go

@@ -20,6 +20,7 @@ import (
 	"context"
 	httputils "github.com/accurics/terrascan/pkg/utils/http"
 	gorillaHandlers "github.com/gorilla/handlers"
+	"go.uber.org/zap"
 	"net/http"
 	"os"
 	"os/signal"
@@ -56,7 +57,7 @@ func (g *APIServer) start(routes []*Route, port, certFile, privateKeyFile string
 		router = mux.NewRouter() // new router
 	)
 
-	logWriter := httputils.GetLogWriter(logger)
+	logWriter := getLogWriter(logger)
 
 	logger.Info("registering routes...")
 
@@ -71,12 +72,12 @@ func (g *APIServer) start(routes []*Route, port, certFile, privateKeyFile string
 	// register all routes
 	for _, v := range routes {
 		logger.Info("Route ", v.verb, " - ", v.path)
-		handler := gorillaHandlers.CustomLoggingHandler(logWriter, http.HandlerFunc(v.fn), httputils.WriteRequestLog)
+		handler := gorillaHandlers.LoggingHandler(logWriter, http.HandlerFunc(v.fn))
 		router.Methods(v.verb).Path(v.path).Handler(handler)
 	}
 
-	router.NotFoundHandler = gorillaHandlers.CustomLoggingHandler(logWriter, http.HandlerFunc(httputils.NotFound), httputils.WriteRequestLog)
-	router.MethodNotAllowedHandler = gorillaHandlers.CustomLoggingHandler(logWriter, http.HandlerFunc(httputils.NotAllowed), httputils.WriteRequestLog)
+	router.NotFoundHandler = gorillaHandlers.LoggingHandler(logWriter, http.HandlerFunc(httputils.NotFound))
+	router.MethodNotAllowedHandler = gorillaHandlers.LoggingHandler(logWriter, http.HandlerFunc(httputils.NotAllowed))
 
 	// Add a route for all static templates / assets. Currently used for the Webhook logs views
 	// go/terrascan/asset is the path where the assets files are located inside the docker container
@@ -122,3 +123,20 @@ func (g *APIServer) start(routes []*Route, port, certFile, privateKeyFile string
 	}
 	logger.Info("server exiting gracefully")
 }
+
+type LogWriter struct {
+	logger *zap.SugaredLogger
+}
+
+// GetLogWriter creates and fetches a LogWriter object
+func getLogWriter(logger *zap.SugaredLogger) LogWriter {
+	return LogWriter{
+		logger: logger,
+	}
+}
+
+// Write writes the byte array to the logger object
+func (l LogWriter) Write(p []byte) (n int, err error) {
+	l.logger.Info(string(p))
+	return len(p), nil
+}