From 8f6639f825141ddc9b3cdb6bdd2be37185a155e3 Mon Sep 17 00:00:00 2001 From: Gemini Wen Date: Tue, 27 Jan 2026 19:26:44 +0800 Subject: [PATCH] fix(response): add nil check for c.Request in error logging Prevents panic when ErrorFrom is called in test contexts where gin.CreateTestContext doesn't set up an HTTP request. Co-Authored-By: Claude Opus 4.5 --- backend/internal/pkg/response/response.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/backend/internal/pkg/response/response.go b/backend/internal/pkg/response/response.go index b491495f..c5b41d6e 100644 --- a/backend/internal/pkg/response/response.go +++ b/backend/internal/pkg/response/response.go @@ -77,7 +77,7 @@ func ErrorFrom(c *gin.Context, err error) bool { statusCode, status := infraerrors.ToHTTP(err) // Log internal errors with full details for debugging - if statusCode >= 500 { + if statusCode >= 500 && c.Request != nil { log.Printf("[ERROR] %s %s\n Error: %s", c.Request.Method, c.Request.URL.Path, err.Error()) }