From 3dcfc6157e3ad3b5dcd8848e3330f15af24a4399 Mon Sep 17 00:00:00 2001
From: mkelvers <mikkelelvers@outlook.com>
Date: Fri, 19 Jun 2026 13:40:05 +0200
Subject: [PATCH] fix: extract requestLogLevel to reduce cyclomatic complexity

---
 internal/server/observability.go | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/internal/server/observability.go b/internal/server/observability.go
index 09e2803..1c88879 100644
--- a/internal/server/observability.go
+++ b/internal/server/observability.go
@@ -23,14 +23,7 @@ func RequestLogger(metrics *observability.Metrics) gin.HandlerFunc {
 		duration := time.Since(start)
 		metrics.ObserveHTTPRequest(c.Request.Method, route, c.Writer.Status(), duration)
 
-		level := observability.LogLevelInfo
 		status := c.Writer.Status()
-		if status >= 500 {
-			level = observability.LogLevelError
-		} else if status >= 400 {
-			level = observability.LogLevelWarn
-		}
-
 		fields := map[string]any{
 			"client_ip":   c.ClientIP(),
 			"duration_ms": float64(duration.Microseconds()) / 1000,
@@ -62,7 +55,7 @@ func RequestLogger(metrics *observability.Metrics) gin.HandlerFunc {
 
 		observability.LogContext(
 			c.Request.Context(),
-			level,
+			requestLogLevel(status),
 			"http_request",
 			"http",
 			c.Request.Method+" "+path,
@@ -71,3 +64,13 @@ func RequestLogger(metrics *observability.Metrics) gin.HandlerFunc {
 		)
 	}
 }
+
+func requestLogLevel(status int) observability.LogLevel {
+	if status >= 500 {
+		return observability.LogLevelError
+	}
+	if status >= 400 {
+		return observability.LogLevelWarn
+	}
+	return observability.LogLevelInfo
+}