From cb405575d15cd46a1ecb0d5fd964b4ab6604d570 Mon Sep 17 00:00:00 2001 From: thguss Date: Sat, 12 Oct 2024 19:21:37 +0900 Subject: [PATCH] fix: updated return to throw --- .../com/smeem/http/aspect/DuplicateRequestAspect.java | 11 ++--------- 1 file changed, 2 insertions(+), 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 index 5e400ba9..e2285748 100644 --- 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 handleDuplicateRequest() { - return ResponseEntity - .status(ExceptionCode.TOO_MANY_REQUESTS.getStatusCode()) - .body(ExceptionResponse.of(ExceptionCode.TOO_MANY_REQUESTS.getMessage() + ": 중복된 요청")); - } }