[BE] refactor: Logback에서 슬랙으로 알람을 보내는 기능을 제거하고, ErrorLoggerConfig를 삭제한다. (#875)

backend/src/main/java/com/festago/common/aop/LogRequestBodyAspect.java

@@ -13,7 +13,6 @@
 import org.aspectj.lang.annotation.Around;
 import org.aspectj.lang.annotation.Aspect;
 import org.aspectj.lang.reflect.MethodSignature;
-import org.slf4j.Logger;
 import org.slf4j.event.Level;
 import org.springframework.http.MediaType;
 import org.springframework.stereotype.Component;
@@ -31,14 +30,12 @@ public class LogRequestBodyAspect {
 
     private final Map<Level, BiConsumer<String, String>> loggerMap = new EnumMap<>(Level.class);
     private final ObjectMapper objectMapper;
-    private final Logger errorLogger;
 
-    public LogRequestBodyAspect(ObjectMapper objectMapper, Logger errorLogger) {
+    public LogRequestBodyAspect(ObjectMapper objectMapper) {
         this.objectMapper = objectMapper;
-        this.errorLogger = errorLogger;
-        loggerMap.put(Level.INFO, this.errorLogger::info);
-        loggerMap.put(Level.WARN, this.errorLogger::warn);
-        loggerMap.put(Level.ERROR, this.errorLogger::error);
+        loggerMap.put(Level.INFO, log::info);
+        loggerMap.put(Level.WARN, log::warn);
+        loggerMap.put(Level.ERROR, log::error);
     }
 
     @Around("@annotation(LogRequestBody)")
@@ -49,7 +46,7 @@ public Object handleAll(ProceedingJoinPoint pjp) throws Throwable {
         Level level = annotation.level();
         ServletRequestAttributes attributes = (ServletRequestAttributes) RequestContextHolder.getRequestAttributes();
 
-        if (attributes == null || !errorLogger.isEnabledForLevel(level)) {
+        if (attributes == null || !log.isEnabledForLevel(level)) {
             return pjp.proceed();
         }
         HttpServletRequest request = attributes.getRequest();

backend/src/main/java/com/festago/common/handler/GlobalExceptionHandler.java

@@ -16,6 +16,7 @@
 import jakarta.servlet.http.HttpServletRequest;
 import lombok.RequiredArgsConstructor;
 import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.springframework.http.HttpHeaders;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.HttpStatusCode;
@@ -31,6 +32,7 @@
 @RequiredArgsConstructor
 public class GlobalExceptionHandler extends ResponseEntityExceptionHandler {
 
+    private static final Logger log = LoggerFactory.getLogger("ErrorLogger");
     private static final String LOG_FORMAT_INFO = "\n[🔵INFO] - ({} {})\n(id: {}, role: {})\n{}\n {}: {}";
     private static final String LOG_FORMAT_WARN = "\n[🟠WARN] - ({} {})\n(id: {}, role: {})";
     private static final String LOG_FORMAT_ERROR = "\n[🔴ERROR] - ({} {})\n(id: {}, role: {})";
@@ -60,7 +62,6 @@ public class GlobalExceptionHandler extends ResponseEntityExceptionHandler {
      */
 
     private final AuthenticateContext authenticateContext;
-    private final Logger errorLogger;
 
     @ExceptionHandler(InvalidMediaTypeException.class)
     public ResponseEntity<ErrorResponse> handle(InvalidMediaTypeException e) {
@@ -133,23 +134,17 @@ protected ResponseEntity<Object> handleMethodArgumentNotValid(
     }
 
     private void logInfo(FestaGoException e, HttpServletRequest request) {
-        if (errorLogger.isInfoEnabled()) {
-            errorLogger.info(LOG_FORMAT_INFO, request.getMethod(), request.getRequestURI(), authenticateContext.getId(),
-                authenticateContext.getRole(), e.getErrorCode(), e.getClass().getName(), e.getMessage());
-        }
+        log.info(LOG_FORMAT_INFO, request.getMethod(), request.getRequestURI(), authenticateContext.getId(),
+            authenticateContext.getRole(), e.getErrorCode(), e.getClass().getName(), e.getMessage());
     }
 
     private void logWarn(FestaGoException e, HttpServletRequest request) {
-        if (errorLogger.isWarnEnabled()) {
-            errorLogger.warn(LOG_FORMAT_WARN, request.getMethod(), request.getRequestURI(),
-                authenticateContext.getId(), authenticateContext.getRole(), e);
-        }
+        log.warn(LOG_FORMAT_WARN, request.getMethod(), request.getRequestURI(),
+            authenticateContext.getId(), authenticateContext.getRole(), e);
     }
 
     private void logError(Exception e, HttpServletRequest request) {
-        if (errorLogger.isErrorEnabled()) {
-            errorLogger.error(LOG_FORMAT_ERROR, request.getMethod(), request.getRequestURI(),
-                authenticateContext.getId(), authenticateContext.getRole(), e);
-        }
+        log.error(LOG_FORMAT_ERROR, request.getMethod(), request.getRequestURI(),
+            authenticateContext.getId(), authenticateContext.getRole(), e);
     }
 }

backend/src/main/resources/logback-spring.xml

@@ -20,20 +20,17 @@
     <include resource="logs/error/file-info-appender.xml"/>
     <include resource="logs/error/file-warn-appender.xml"/>
     <include resource="logs/error/file-error-appender.xml"/>
-    <include resource="logs/slack/slack-error-appender.xml"/>
 
     <logger additivity="false" level="INFO" name="ErrorLogger">
       <appender-ref ref="ERROR_ERROR"/>
       <appender-ref ref="ERROR_WARN"/>
       <appender-ref ref="ERROR_INFO"/>
-      <appender-ref ref="ASYNC_SLACK_ERROR"/>
     </logger>
 
     <root level="INFO">
       <appender-ref ref="FILE-INFO"/>
       <appender-ref ref="FILE-WARN"/>
       <appender-ref ref="FILE-ERROR"/>
-      <appender-ref ref="ASYNC_SLACK_ERROR"/>
     </root>
   </springProfile>
 
@@ -44,22 +41,17 @@
     <include resource="logs/error/file-info-appender.xml"/>
     <include resource="logs/error/file-warn-appender.xml"/>
     <include resource="logs/error/file-error-appender.xml"/>
-    <include resource="logs/slack/slack-error-appender.xml"/>
-    <include resource="logs/slack/slack-warn-appender.xml"/>
 
     <logger additivity="false" level="INFO" name="ErrorLogger">
       <appender-ref ref="ERROR_ERROR"/>
       <appender-ref ref="ERROR_WARN"/>
       <appender-ref ref="ERROR_INFO"/>
-      <appender-ref ref="ASYNC_SLACK_ERROR"/>
-      <appender-ref ref="ASYNC_SLACK_WARN"/>
     </logger>
 
     <root level="INFO">
       <appender-ref ref="FILE-INFO"/>
       <appender-ref ref="FILE-WARN"/>
       <appender-ref ref="FILE-ERROR"/>
-      <appender-ref ref="ASYNC_SLACK_ERROR"/>
     </root>
   </springProfile>
 

backend/src/test/java/com/festago/support/CustomWebMvcTest.java

@@ -1,7 +1,6 @@
 package com.festago.support;
 
 import com.festago.common.aop.ValidPageableAspect;
-import com.festago.config.ErrorLoggerConfig;
 import java.lang.annotation.Retention;
 import java.lang.annotation.RetentionPolicy;
 import org.springframework.boot.test.autoconfigure.web.servlet.WebMvcTest;
@@ -12,8 +11,7 @@
 
 @EnableAspectJAutoProxy
 @WebMvcTest
-@Import({TestAuthConfig.class, ErrorLoggerConfig.class, MockAllServiceBeanFactoryPostProcessor.class,
-    ValidPageableAspect.class})
+@Import({TestAuthConfig.class, MockAllServiceBeanFactoryPostProcessor.class, ValidPageableAspect.class})
 @Retention(RetentionPolicy.RUNTIME)
 @TestExecutionListeners(value = {MockAuthTestExecutionListener.class,
     ResetMockTestExecutionListener.class}, mergeMode = MergeMode.MERGE_WITH_DEFAULTS)