refactor: status code >= 500 is treated as an error

fufuok · Feb 21, 2024 · f5c25b3 · f5c25b3
1 parent ee225b2
commit f5c25b3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/web/fiber/middleware/counter.go b/web/fiber/middleware/counter.go
@@ -36,7 +36,7 @@ func HTTPCounter(name string) fiber.Handler {
 
 		err = c.Next()
 
-		if err != nil || c.Response().StatusCode() >= fiber.StatusBadRequest {
+		if err != nil || c.Response().StatusCode() >= fiber.StatusInternalServerError {
 			counter.Err.Add(1)
 		} else {
 			counter.OK.Add(1)

diff --git a/web/gin/middleware/counter.go b/web/gin/middleware/counter.go
@@ -2,6 +2,7 @@ package middleware
 
 import (
 	"fmt"
+	"net/http"
 	"sync/atomic"
 	"time"
 
@@ -36,7 +37,7 @@ func HTTPCounter(name string) gin.HandlerFunc {
 
 		c.Next()
 
-		if len(c.Errors) > 0 || c.Writer.Status() >= 400 {
+		if len(c.Errors) > 0 || c.Writer.Status() >= http.StatusInternalServerError {
 			counter.Err.Add(1)
 		} else {
 			counter.OK.Add(1)