From ae4b37bb136397ca4631ed39526b8156622f285b Mon Sep 17 00:00:00 2001
From: Marc Pervaz Boocha <mboocha@sudomsg.com>
Date: Thu, 7 Aug 2025 22:54:36 +0530
Subject: Removed the extra logger arguement from http

---
 logging/http/http.go | 19 ++++---------------
 1 file changed, 4 insertions(+), 15 deletions(-)

(limited to 'logging/http/http.go')

diff --git a/logging/http/http.go b/logging/http/http.go
index b88ae3a..04b2e0c 100644
--- a/logging/http/http.go
+++ b/logging/http/http.go
@@ -24,7 +24,6 @@ type tracingWriter struct {
 
 var (
 	_ http.ResponseWriter                       = &tracingWriter{}
-	_ http.Flusher                              = &tracingWriter{}
 	_ http.Hijacker                             = &tracingWriter{}
 	_ interface{ Unwrap() http.ResponseWriter } = &tracingWriter{}
 )
@@ -43,11 +42,6 @@ func (tw *tracingWriter) Write(b []byte) (int, error) {
 	return n, err
 }
 
-func (tw *tracingWriter) Flush() {
-	rc := http.NewResponseController(tw.ResponseWriter)
-	_ = rc.Flush()
-}
-
 func (tw *tracingWriter) Hijack() (net.Conn, *bufio.ReadWriter, error) {
 	rc := http.NewResponseController(tw.ResponseWriter)
 	conn, rw, err := rc.Hijack()
@@ -66,8 +60,7 @@ func (tw *tracingWriter) Unwrap() http.ResponseWriter {
 // It recovers panics during request processing, logs them with stack traces,
 // and returns a 500 Internal Server Error if the connection is not hijacked.
 type LoggingHandler struct {
-	next   http.Handler
-	logger *slog.Logger
+	next http.Handler
 }
 
 var _ http.Handler = &LoggingHandler{}
@@ -75,10 +68,9 @@ var _ http.Handler = &LoggingHandler{}
 // New creates a new LoggingHandler wrapping the next http.Handler, using the provided slog.Logger.
 //
 // If logger is nil, it attempts to obtain one from the request context.
-func New(next http.Handler, logger *slog.Logger) *LoggingHandler {
+func New(next http.Handler) *LoggingHandler {
 	return &LoggingHandler{
-		next:   next,
-		logger: logger,
+		next: next,
 	}
 }
 
@@ -90,10 +82,7 @@ func New(next http.Handler, logger *slog.Logger) *LoggingHandler {
 func (h *LoggingHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
 	ctx := r.Context()
 
-	logger := h.logger
-	if logger == nil {
-		logger = logging.FromContext(ctx)
-	}
+	logger := logging.FromContext(ctx)
 
 	logger = logger.With(
 		slog.String("method", r.Method),
-- 
cgit v1.2.3-70-g09d2