fix: updated return to throw

Team-Smeme · Oct 12, 2024 · cb40557 · cb40557
1 parent 6ccbfd3
commit cb40557
Showing 1 changed file with 2 additions and 9 deletions.
diff --git a/smeem-input-http/src/main/java/com/smeem/http/aspect/DuplicateRequestAspect.java b/smeem-input-http/src/main/java/com/smeem/http/aspect/DuplicateRequestAspect.java
@@ -1,12 +1,11 @@
 package com.smeem.http.aspect;
 
 import com.smeem.common.exception.ExceptionCode;
-import com.smeem.http.controller.dto.ExceptionResponse;
+import com.smeem.common.exception.SmeemException;
 import lombok.val;
 import org.aspectj.lang.ProceedingJoinPoint;
 import org.aspectj.lang.annotation.Around;
 import org.aspectj.lang.annotation.Aspect;
-import org.springframework.http.ResponseEntity;
 import org.springframework.stereotype.Component;
 import org.springframework.web.context.request.RequestContextHolder;
 import org.springframework.web.context.request.ServletRequestAttributes;
@@ -33,7 +32,7 @@ public Object duplicateRequestCheck(ProceedingJoinPoint joinPoint) throws Throwa
         val requestId = joinPoint.getSignature().toLongString();
         if (requestSet.contains(requestId)) {
             // 중복 요청인 경우
-            return handleDuplicateRequest();
+           throw new SmeemException(ExceptionCode.TOO_MANY_REQUESTS);
         }
         requestSet.add(requestId);
 
@@ -43,10 +42,4 @@ public Object duplicateRequestCheck(ProceedingJoinPoint joinPoint) throws Throwa
             requestSet.remove(requestId);
         }
     }
-
-    private ResponseEntity<ExceptionResponse> handleDuplicateRequest() {
-        return ResponseEntity
-                .status(ExceptionCode.TOO_MANY_REQUESTS.getStatusCode())
-                .body(ExceptionResponse.of(ExceptionCode.TOO_MANY_REQUESTS.getMessage() + ": 중복된 요청"));
-    }
 }