update rate limitter to use APIError

backend/internal/middleware/rate_limit.go

@@ -1,6 +1,7 @@
 package middleware
 
 import (
+	"KonferCA/SPUR/internal/v1/v1_common"
 	"net/http"
 	"sync"
 	"time"
@@ -122,9 +123,11 @@ func (rl *RateLimiter) RateLimit() echo.MiddlewareFunc {
 					Dur("remaining", remaining).
 					Msg("request blocked: rate limit exceeded")
 
-				return echo.NewHTTPError(
+				return v1_common.Fail(
+					c,
 					http.StatusTooManyRequests,
 					"too many requests, please try again in "+remaining.Round(time.Second).String(),
+					nil,
 				)
 			}
 
@@ -151,9 +154,11 @@ func (rl *RateLimiter) RateLimit() echo.MiddlewareFunc {
 					Dur("block_duration", blockDuration).
 					Msg("IP blocked: rate limit exceeded")
 
-				return echo.NewHTTPError(
+				return v1_common.Fail(
+					c,
 					http.StatusTooManyRequests,
 					"too many requests, please try again in "+blockDuration.Round(time.Second).String(),
+					nil,
 				)
 			}
 

backend/internal/tests/rate_limit_test.go

@@ -7,6 +7,7 @@ import (
 	"time"
 
 	"KonferCA/SPUR/internal/middleware"
+	"KonferCA/SPUR/internal/v1/v1_common"
 
 	"github.com/labstack/echo/v4"
 	"github.com/stretchr/testify/assert"
@@ -69,7 +70,7 @@ func TestRateLimiter(t *testing.T) {
 		rec = httptest.NewRecorder()
 		c = e.NewContext(req, rec)
 		err = h(c)
-		he, ok := err.(*echo.HTTPError)
+		he, ok := err.(*v1_common.APIError)
 		assert.True(t, ok)
 		assert.Equal(t, http.StatusTooManyRequests, he.Code)
 	})
@@ -103,7 +104,7 @@ func TestRateLimiter(t *testing.T) {
 			rec = httptest.NewRecorder()
 			c = e.NewContext(req, rec)
 			err = h(c)
-			he, ok := err.(*echo.HTTPError)
+			he, ok := err.(*v1_common.APIError)
 			assert.True(t, ok)
 			assert.Equal(t, http.StatusTooManyRequests, he.Code)
 		}
@@ -137,7 +138,7 @@ func TestRateLimiter(t *testing.T) {
 		c = e.NewContext(req, rec)
 		err = h(c)
 		assert.Error(t, err)
-		he, ok := err.(*echo.HTTPError)
+		he, ok := err.(*v1_common.APIError)
 		assert.True(t, ok)
 		assert.Equal(t, http.StatusTooManyRequests, he.Code)
 
@@ -179,7 +180,7 @@ func TestRateLimiter(t *testing.T) {
 			c = e.NewContext(req, rec)
 			err = h(c)
 			assert.Error(t, err)
-			he, ok := err.(*echo.HTTPError)
+			he, ok := err.(*v1_common.APIError)
 			assert.True(t, ok)
 			assert.Equal(t, http.StatusTooManyRequests, he.Code)