CLAP-196 Fix: 요청 실패 시 설명 구체화

Author: parkjaehak

src/main/java/clap/server/application/port/inbound/log/CreateAnonymousLogsUsecase.java

@@ -7,6 +7,6 @@
 import java.time.LocalDateTime;
 
 public interface CreateAnonymousLogsUsecase {
-    void createAnonymousLog(HttpServletRequest request, int statusCode, LogStatus logStatus,  Object responseBody, String requestBody, String nickName);
+    void createAnonymousLog(HttpServletRequest request, int statusCode, String customCode, LogStatus logStatus,  Object responseBody, String requestBody, String nickName);
 
 }

src/main/java/clap/server/application/port/inbound/log/CreateMemberLogsUsecase.java

@@ -8,5 +8,5 @@
 
 public interface CreateMemberLogsUsecase {
 
-    void createMemberLog(HttpServletRequest request, int statusCode, LogStatus logStatus, Object responseBody, String requestBody, Long userId);
+    void createMemberLog(HttpServletRequest request, int statusCode,String customCode, LogStatus logStatus, Object responseBody, String requestBody, Long userId);
 }

src/main/java/clap/server/application/service/log/CreateAnonymousLogsService.java

@@ -18,8 +18,8 @@ public class CreateAnonymousLogsService implements CreateAnonymousLogsUsecase {
     private final ApiLogPersistenceAdapter apiLogPersistenceAdapter;
 
     @Override
-    public void createAnonymousLog(HttpServletRequest request,  int statusCode,  LogStatus logStatus, Object responseBody, String requestBody, String nickName) {
-        AnonymousLog anonymousLog = AnonymousLog.createAnonymousLog(request, statusCode, logStatus, responseBody, requestBody, nickName);
+    public void createAnonymousLog(HttpServletRequest request,  int statusCode, String customCode, LogStatus logStatus, Object responseBody, String requestBody, String nickName) {
+        AnonymousLog anonymousLog = AnonymousLog.createAnonymousLog(request, statusCode,customCode, logStatus, responseBody, requestBody, nickName);
         apiLogPersistenceAdapter.saveAnonymousLog(anonymousLog);
     }
 }

src/main/java/clap/server/application/service/log/CreateMemberLogsService.java

@@ -23,9 +23,9 @@ public class CreateMemberLogsService implements CreateMemberLogsUsecase {
 
     @Override
     @Transactional
-    public void createMemberLog(HttpServletRequest request, int statusCode, LogStatus logStatus, Object responseBody, String requestBody, Long userId) {
+    public void createMemberLog(HttpServletRequest request, int statusCode, String customCode,LogStatus logStatus, Object responseBody, String requestBody, Long userId) {
         Member member = memberService.findById(userId);
-        MemberLog memberLog = MemberLog.createMemberLog(request, statusCode, logStatus,  responseBody, requestBody, member);
+        MemberLog memberLog = MemberLog.createMemberLog(request, statusCode, customCode, logStatus,  responseBody, requestBody, member);
         apiLogPersistenceAdapter.saveMemberLog(memberLog);
     }
 }

src/main/java/clap/server/config/aop/LoggingAspect.java

@@ -62,9 +62,11 @@ public Object logApiRequests(ProceedingJoinPoint joinPoint) throws Throwable {
         } finally {
             LogStatus logStatus = getLogType((MethodSignature) joinPoint.getSignature());
             int statusCode;
+            String customCode = null;
             if (capturedException != null) {
                 if (capturedException instanceof BaseException e) {
                     statusCode = e.getCode().getHttpStatus().value();
+                    customCode = e.getCode().getCustomCode();
                 } else {
                     ModelAndView modelAndView = handlerExceptionResolver.resolveException(request, response, null, capturedException);
                     statusCode = modelAndView.getStatus().value();
@@ -75,14 +77,14 @@ public Object logApiRequests(ProceedingJoinPoint joinPoint) throws Throwable {
 
             if (logStatus != null) {
                 if (LogStatus.LOGIN.equals(logStatus)) {
-                    createAnonymousLogsUsecase.createAnonymousLog(request, statusCode, logStatus, result, getRequestBody(request), getNicknameFromRequestBody(request));
+                    createAnonymousLogsUsecase.createAnonymousLog(request, statusCode, customCode, logStatus, result, getRequestBody(request), getNicknameFromRequestBody(request));
                 } else {
                     if (!isUserAuthenticated()) {
                         log.error("로그인 시도 로그를 기록할 수 없음");
                     } else {
                         Object principal = SecurityContextHolder.getContext().getAuthentication().getPrincipal();
                         if (principal instanceof SecurityUserDetails userDetails) {
-                            createMemberLogsUsecase.createMemberLog(request, statusCode, logStatus, result, getRequestBody(request), userDetails.getUserId());
+                            createMemberLogsUsecase.createMemberLog(request, statusCode, customCode, logStatus, result, getRequestBody(request), userDetails.getUserId());
                         }
                     }
                 }

src/main/java/clap/server/domain/model/log/AnonymousLog.java

@@ -16,13 +16,13 @@
 public class AnonymousLog extends ApiLog {
     private String loginNickname;
 
-    public static AnonymousLog createAnonymousLog(HttpServletRequest request, int statusCode,  LogStatus logStatus, Object responseBody, String requestBody, String nickName) {
+    public static AnonymousLog createAnonymousLog(HttpServletRequest request, int statusCode, String customCode, LogStatus logStatus, Object responseBody, String requestBody, String nickName) {
         return AnonymousLog.builder()
                 .clientIp(ClientIpParseUtil.getClientIp(request))
                 .requestUrl(request.getRequestURI())
                 .requestMethod(request.getMethod())
                 .statusCode(statusCode)
-                .customStatusCode("")
+                .customStatusCode(customCode != null ? customCode : "")
                 .requestBody(requestBody)
                 .responseBody(responseBody != null ? responseBody.toString() : "로그인 실패")
                 .requestAt(LocalDateTime.now())

src/main/java/clap/server/domain/model/log/MemberLog.java

@@ -17,15 +17,15 @@
 public class MemberLog extends ApiLog {
     private Member member;
 
-    public static MemberLog createMemberLog(HttpServletRequest request, int statusCode, LogStatus logStatus, Object responseBody, String requestBody, Member member) {
+    public static MemberLog createMemberLog(HttpServletRequest request, int statusCode, String customCode, LogStatus logStatus, Object responseBody, String requestBody, Member member) {
         return MemberLog.builder()
                 .clientIp(ClientIpParseUtil.getClientIp(request))
                 .requestUrl(request.getRequestURI())
                 .requestMethod(request.getMethod())
                 .statusCode(statusCode)
-                .customStatusCode("")
+                .customStatusCode(customCode != null ? customCode : "")
                 .requestBody(requestBody)
-                .responseBody(responseBody != null ? responseBody.toString() : "일반 실패")
+                .responseBody(responseBody != null ? responseBody.toString() : logStatus.getDescription() + " 실패")
                 .requestAt(LocalDateTime.now())
                 .logStatus(logStatus)
                 .member(member)