Skip to content

Commit fee6bf6

Browse files
committed
Fix warning header
1 parent b3360ee commit fee6bf6

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

api/pkg/middlewares/http_request_logger_middleware.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ package middlewares
22

33
import (
44
"fmt"
5+
"slices"
56

67
"github.com/NdoleStudio/httpsms/pkg/telemetry"
78
"github.com/gofiber/fiber/v2"
@@ -22,8 +23,8 @@ func HTTPRequestLogger(tracer telemetry.Tracer, logger telemetry.Logger) fiber.H
2223

2324
statusCode := c.Response().StatusCode()
2425
span.AddEvent(fmt.Sprintf("finished handling request with traceID: [%s], statusCode: [%d]", span.SpanContext().TraceID().String(), statusCode))
25-
if statusCode >= 300 && len(c.Request().Body()) > 0 && statusCode != 401 {
26-
ctxLogger.Warn(stacktrace.NewError(fmt.Sprintf("http.status [%d], body [%s]", statusCode, string(c.Request().Body()))))
26+
if statusCode >= 300 && len(c.Request().Body()) > 0 && !slices.Contains([]int{401, 402}, statusCode) {
27+
ctxLogger.WithString("client.version", c.Get(clientVersionHeader)).Warn(stacktrace.NewError(fmt.Sprintf("http.status [%d], body [%s]", statusCode, string(c.Request().Body()))))
2728
}
2829

2930
return response

0 commit comments

Comments
 (0)