From af7b5d9478df3da1a46e14e39fafb9fe475b0c0f Mon Sep 17 00:00:00 2001
From: Alan Pearce
Date: Sun, 19 May 2024 15:53:09 +0200
Subject: use zap for logging http requests as well

---
 internal/server/logging.go | 21 +++++++++------------
 1 file changed, 9 insertions(+), 12 deletions(-)

(limited to 'internal/server/logging.go')

diff --git a/internal/server/logging.go b/internal/server/logging.go
index 135f06e..fcdab18 100644
--- a/internal/server/logging.go
+++ b/internal/server/logging.go
@@ -1,9 +1,8 @@
 package server
 
 import (
-	"fmt"
-	"io"
 	"net/http"
+	"website/internal/log"
 )
 
 type loggingResponseWriter struct {
@@ -25,7 +24,6 @@ func NewLoggingResponseWriter(w http.ResponseWriter) *loggingResponseWriter {
 
 type wrappedHandlerOptions struct {
 	defaultHostname string
-	logger          io.Writer
 }
 
 func wrapHandlerWithLogging(wrappedHandler http.Handler, opts wrappedHandlerOptions) http.Handler {
@@ -41,15 +39,14 @@ func wrapHandlerWithLogging(wrappedHandler http.Handler, opts wrappedHandlerOpti
 		lw := NewLoggingResponseWriter(w)
 		wrappedHandler.ServeHTTP(lw, r)
 		statusCode := lw.statusCode
-		fmt.Fprintf(
-			opts.logger,
-			"%s %s %d %s %s %s\n",
-			scheme,
-			r.Method,
-			statusCode,
-			host,
-			r.URL.Path,
-			lw.Header().Get("Location"),
+		log.Info(
+			"http request",
+			"scheme", scheme,
+			"method", r.Method,
+			"status", statusCode,
+			"host", host,
+			"path", r.URL.Path,
+			"location", lw.Header().Get("Location"),
 		)
 	})
 }
-- 
cgit 1.4.1