Merge pull request #126 from WE-ARE-RACCOONS/develop

RAC-315 refactor : ERROR 로그 추가
WE-ARE-RACCOONS · Feb 22, 2024 · 525fcbd · 525fcbd
2 parents 42c4652 + 038d7b0
commit 525fcbd
Show file tree

Hide file tree

Showing 6 changed files with 13 additions and 5 deletions.
diff --git a/...in/java/com/postgraduate/global/config/security/jwt/filter/CustomAccessDeniedHandler.java b/...in/java/com/postgraduate/global/config/security/jwt/filter/CustomAccessDeniedHandler.java
@@ -7,6 +7,7 @@
 import jakarta.servlet.http.HttpServletRequest;
 import jakarta.servlet.http.HttpServletResponse;
 import lombok.RequiredArgsConstructor;
+import lombok.extern.slf4j.Slf4j;
 import org.springframework.beans.factory.annotation.Value;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.MediaType;
@@ -21,6 +22,7 @@
 
 @Component
 @RequiredArgsConstructor
+@Slf4j
 public class CustomAccessDeniedHandler implements AccessDeniedHandler {
     @Value("${log.Type}")
     private String env;
@@ -34,6 +36,7 @@ public void handle(HttpServletRequest request, HttpServletResponse response, Acc
         response.setContentType(MediaType.APPLICATION_JSON_VALUE);
         response.setCharacterEncoding("UTF-8");
         logService.save(new LogRequest(env, CustomAccessDeniedHandler.class.getSimpleName(), PERMISSION_DENIED.getMessage()));
+        log.error("AccessDenied : {}", PERMISSION_DENIED.getMessage());
         objectMapper.writeValue(
                 response.getOutputStream(),
                 new ErrorResponse(AUTH_DENIED.getCode(), PERMISSION_DENIED.getMessage())

diff --git a/...va/com/postgraduate/global/config/security/jwt/filter/CustomAuthenticationEntryPoint.java b/...va/com/postgraduate/global/config/security/jwt/filter/CustomAuthenticationEntryPoint.java
@@ -8,6 +8,7 @@
 import jakarta.servlet.http.HttpServletRequest;
 import jakarta.servlet.http.HttpServletResponse;
 import lombok.RequiredArgsConstructor;
+import lombok.extern.slf4j.Slf4j;
 import org.springframework.beans.factory.annotation.Value;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.MediaType;
@@ -22,6 +23,7 @@
 
 @Component
 @RequiredArgsConstructor
+@Slf4j
 public class CustomAuthenticationEntryPoint implements AuthenticationEntryPoint {
     @Value("${log.Type}")
     private String env;
@@ -35,6 +37,7 @@ public void commence(HttpServletRequest request, HttpServletResponse response, A
         response.setContentType(MediaType.APPLICATION_JSON_VALUE);
         response.setCharacterEncoding("UTF-8");
         logService.save(new LogRequest(env, CustomAuthenticationEntryPoint.class.getSimpleName(), FAILED_AUTH.getMessage()));
+        log.error("AuthenticationEntryPoint : {}", FAILED_AUTH.getMessage());
         objectMapper.writeValue(
                 response.getOutputStream(),
                 new ErrorResponse(AUTH_FAILED.getCode(), FAILED_AUTH.getMessage())

diff --git a/src/main/java/com/postgraduate/global/config/security/jwt/util/JwtUtils.java b/src/main/java/com/postgraduate/global/config/security/jwt/util/JwtUtils.java
@@ -155,6 +155,7 @@ private void jwtExceptionHandler(HttpStatus status, HttpServletResponse response
         response.setStatus(status.value());
         response.setContentType(CONTENT_TYPE);
         response.setCharacterEncoding(CHARACTER_ENCODING);
+        log.error("errorCode {}, errorMessage {}", ex.getErrorCode(), ex.getMessage());
         try {
             logService.save(new LogRequest(env, JwtFilter.class.getSimpleName(), ex.getMessage()));
             String json = new ObjectMapper().writeValueAsString(ResponseDto.create(ex.getErrorCode(), ex.getMessage()));

diff --git a/src/main/java/com/postgraduate/global/logging/aop/LogAspect.java b/src/main/java/com/postgraduate/global/logging/aop/LogAspect.java
@@ -50,13 +50,15 @@ private Object getObject(ProceedingJoinPoint joinPoint) throws Throwable {
             return result;
         }catch (ApplicationException e) {
             if (traceStatus != null) {
-                logTrace.exception(e, traceStatus);
+                log.error("ErrorCode {} errorMessage {}",e.getErrorCode(), e.getMessage());
+                log.error("{}", e.getStackTrace());
                 logService.save(new LogRequest(env, traceStatus.threadId(), traceStatus.methodName(), e.getMessage()));
             }
             throw e;
         }catch (Exception e) {
             if (traceStatus != null) {
-                logTrace.exception(e, traceStatus);
+                log.error("500 ERROR {}", e.getMessage());
+                log.error("{}", e.getStackTrace());
                 logService.save(new LogRequest(env, traceStatus.threadId(), traceStatus.methodName(), e.getMessage()));
             }
             throw e;

diff --git a/src/main/java/com/postgraduate/global/logging/service/LogService.java b/src/main/java/com/postgraduate/global/logging/service/LogService.java
@@ -23,8 +23,7 @@ public class LogService {
     @Value("${log.uri}")
     private String logUri;
 
-    public void save(LogRequest logRequest) throws IOException {
-        log.info("log save");
+    public void save(LogRequest logRequest) {
         webClient.post()
                 .uri(logUri)
                 .accept(MediaType.APPLICATION_JSON)

diff --git a/src/main/resources/logback-spring.xml b/src/main/resources/logback-spring.xml
@@ -64,7 +64,7 @@
             <timeBasedFileNamingAndTriggeringPolicy class="ch.qos.logback.core.rolling.SizeAndTimeBasedFNATP">
                 <maxFileSize>50MB</maxFileSize>
             </timeBasedFileNamingAndTriggeringPolicy>
-            <maxHistory>30</maxHistory>
+            <maxHistory>14</maxHistory>
         </rollingPolicy>
     </appender>