Default to 200OK, otherwise get the HTTP return status

quay · Sep 15, 2020 · 1f0619f · 1f0619f
1 parent 3c87d98
commit 1f0619f
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/httptransport/logginghandler.go b/httptransport/logginghandler.go
@@ -10,7 +10,6 @@ import (
 
 type httpStatusWriter struct {
 	http.ResponseWriter
-
 	StatusCode int
 }
 
@@ -21,13 +20,16 @@ func LoggingHandler(next http.Handler) http.HandlerFunc {
 
 		log := zerolog.Ctx(r.Context())
 
+		// default HTTP StatusOK
+		lrw := &httpStatusWriter{ResponseWriter: w, StatusCode: http.StatusOK}
+
 		next.ServeHTTP(w, r)
 
 		log.Info().
 			Str("remote addr", r.RemoteAddr).
 			Str("method", r.Method).
 			Str("request uri", r.RequestURI).
-			Str("status", strconv.Itoa(http.StatusOK)).
+			Str("status", strconv.Itoa(lrw.StatusCode)).
 			Float64("elapsed time (md)", float64(time.Since(start).Nanoseconds())*1e-6).
 			Msg("handled HTTP request")
 	}