feat: logging 기능 & slack notification 리팩토링 (#294)

* refactor: 커스텀 에러 메시지가 보이도록 수정

* refactor: 로그 처리중 slack알림 보내도록 설정

* refactor: 중복 코드 제거

---------

Co-authored-by: 나용준 <141994188+youngJun99@users.noreply.github.com>

Author: yongjun0511

src/main/java/ssu/eatssu/domain/slack/entity/SlackMessageFormat.java

@@ -73,23 +73,46 @@ public static String sendUserInquiry(Inquiry inquiry) {
         return messageFormat.format(args);
     }
 
-
-    public static String sendServerError(BaseException ex) {
-        MessageFormat messageFormat = new MessageFormat(
-                """
+    public static String sendServerError(Throwable ex, String method, String uri, String userId, String args) {
+        final String messageTemplate = """
                         ===================
                         *서버 에러 발생*
-                        - 예외 상태코드: {0}
-                        - 예외 메시지: {1}
-                        - 개발환경: {2}
+                        - {0}: {1}
+                        - 예외 메시지: {2}
+                        - 개발환경: {3}
+                        *요청 정보*
+                        - HTTP Method: {4}
+                        - URI: {5}
+                        - User ID: {6}
+                        - 요청 파라미터: {7}
                         ===================
-                        """
-        );
-        Object[] args = {
-                ex.getStatus(),
-                ex.getStatus().getMessage(),
+                        """;
+        MessageFormat messageFormat = new MessageFormat(messageTemplate);
+
+        String errorTypeLabel;
+        Object errorTypeValue;
+        String errorMessage;
+
+        if (ex instanceof BaseException baseException) {
+            errorTypeLabel = "예외 상태코드";
+            errorTypeValue = baseException.getStatus();
+            errorMessage = baseException.getStatus().getMessage();
+        } else {
+            errorTypeLabel = "예외 타입";
+            errorTypeValue = ex.getClass().getSimpleName();
+            errorMessage = ex.getMessage() != null ? ex.getMessage() : "메시지 없음";
+        }
+
+        Object[] formatArgs = {
+                errorTypeLabel,
+                errorTypeValue,
+                errorMessage,
                 serverEnv,
+                method,
+                uri,
+                userId,
+                args != null && args.length() > 500 ? args.substring(0, 500) + "...(truncated)" : args
         };
-        return messageFormat.format(args);
+        return messageFormat.format(formatArgs);
     }
 }

src/main/java/ssu/eatssu/domain/slack/service/SlackErrorNotifier.java

@@ -18,12 +18,12 @@ public SlackErrorNotifier(SlackService slackService) {
         this.slackService = slackService;
     }
 
-    public void notify(BaseException ex) {
+    public void notify(Throwable ex, String method, String uri, String userId, String args) {
         if (!"prod".equals(serverEnv)) {
             return;
         }
         try {
-            String message = SlackMessageFormat.sendServerError(ex);
+            String message = SlackMessageFormat.sendServerError(ex, method, uri, userId, args);
             slackService.sendSlackMessage(message, SlackChannel.SERVER_ERROR);
         } catch (Exception slackEx) {
             log.warn("슬랙 전송 실패: {}", slackEx.getMessage());

src/main/java/ssu/eatssu/global/handler/GlobalExceptionHandler.java

@@ -46,15 +46,11 @@ public class GlobalExceptionHandler extends ResponseEntityExceptionHandler {
      */
     @ExceptionHandler(BaseException.class)
     public ResponseEntity<BaseResponse<Void>> handleBaseException(BaseException e) {
-        if (BaseResponseStatus.sendSlackNotification(e.getStatus())) {
-            slackErrorNotifier.notify(e);
-        }
         return ResponseEntity.status(e.getStatus().getHttpStatus()).body(BaseResponse.fail(e.getStatus()));
     }
 
     @ExceptionHandler(Exception.class)
     public ResponseEntity<BaseResponse<Void>> handleAllUnhandledException(Exception ex) {
-        slackErrorNotifier.notify(new BaseException(BaseResponseStatus.BAD_REQUEST));
         return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR)
                              .body(BaseResponse.fail(BaseResponseStatus.INTERNAL_SERVER_ERROR));
     }
@@ -241,10 +237,6 @@ protected ResponseEntity<Object> handleExceptionInternal(
 
         HttpStatus status = HttpStatus.valueOf(statusCode.value());
 
-        if (status.is5xxServerError()) {
-            slackErrorNotifier.notify(new BaseException(BaseResponseStatus.BAD_REQUEST));
-        }
-
         BaseResponseStatus responseStatus = status.is4xxClientError()
                 ? BaseResponseStatus.BAD_REQUEST
                 : BaseResponseStatus.INTERNAL_SERVER_ERROR;

src/main/java/ssu/eatssu/global/log/ControllerLogAspect.java

@@ -14,6 +14,9 @@
 import org.springframework.web.context.request.RequestContextHolder;
 import org.springframework.web.context.request.ServletRequestAttributes;
 import ssu.eatssu.domain.auth.security.CustomUserDetails;
+import ssu.eatssu.domain.slack.service.SlackErrorNotifier;
+import ssu.eatssu.global.handler.response.BaseException;
+import ssu.eatssu.global.handler.response.BaseResponseStatus;
 import ssu.eatssu.global.log.annotation.LogMask;
 
 import java.lang.reflect.Field;
@@ -30,6 +33,7 @@
 public class ControllerLogAspect {
 
     private final ObjectMapper objectMapper;
+    private final SlackErrorNotifier slackErrorNotifier;
 
     @Pointcut("within(@org.springframework.web.bind.annotation.RestController *)")
     public void restController() {}
@@ -48,14 +52,16 @@ public Object logApi(ProceedingJoinPoint joinPoint) throws Throwable {
         Object[] args = joinPoint.getArgs();
 
         // 요청자
-        String userIdLog = IntStream.range(0, args.length)
+        String userId = IntStream.range(0, args.length)
                 .filter(i -> args[i] instanceof CustomUserDetails)
                 .mapToObj(i -> {
                     CustomUserDetails user = (CustomUserDetails) args[i];
-                    return "userId=" + user.getId();
+                    return String.valueOf(user.getId());
                 })
                 .findFirst()
-                .orElse("userId=anonymous");
+                .orElse("anonymous");
+
+        String userIdLog = "userId=" + userId;
 
         // 나머지 요청 인자
         String otherArgsJson = IntStream.range(0, args.length)
@@ -103,11 +109,24 @@ public Object logApi(ProceedingJoinPoint joinPoint) throws Throwable {
             return result;
         } catch (Throwable e) {
             long time = System.currentTimeMillis() - start;
-            log.error("EXCEPTION {} {} ({} ms) cause={}", method, uri, time, e.getMessage(), e);
+            String causeMessage = getCauseMessage(e);
+            String exceptionType = e.getClass().getSimpleName();
+            log.error("EXCEPTION {} {} ({} ms) type={} cause={}", method, uri, time, exceptionType, causeMessage, e);
+
+            slackErrorNotifier.notify(e, method, uri, userId, argsJson);
             throw e;
         }
     }
 
+    private String getCauseMessage(Throwable e) {
+        if (e instanceof BaseException) {
+            BaseException baseException = (BaseException) e;
+            return baseException.getStatus().getMessage();
+        }
+        String message = e.getMessage();
+        return message != null ? message : e.getClass().getSimpleName();
+    }
+
     private Map<String, Object> toSafeMap(Object arg) {
         Map<String, Object> result = new HashMap<>();
         for (Field field : arg.getClass().getDeclaredFields()) {