fix: reflect on 지윤's code review (#9, #10)

1. ErrorDTO에 detail 부분 삭제
2. 에러 전, log.warn 추가
3. DTO 타입 Record 타입으로 통일

src/main/java/com/compono/ibackend/common/dto/error/response/ErrorDTO.java

@@ -3,28 +3,19 @@
 import com.compono.ibackend.common.enumType.ErrorCode;
 import com.compono.ibackend.common.exception.CustomException;
 import lombok.Builder;
-import lombok.Data;
 import org.springframework.http.ResponseEntity;
 
-@Data
 @Builder
-public class ErrorDTO {
-
-    private int code;
-    private String msg;
-    private String detail;
+public record ErrorDTO(int code, String msg) {
 
     public static ResponseEntity<ErrorDTO> toResponseEntity(CustomException ex) {
         ErrorCode errorType = ex.getErrorCode();
-        String detail = ex.getDetail();
 
         return ResponseEntity
                 .status(ex.getStatus())
                 .body(ErrorDTO.builder()
                         .code(errorType.getCode())
                         .msg(errorType.getMsg())
-                        .detail(detail)
                         .build());
     }
-}
-
+}

src/main/java/com/compono/ibackend/common/exception/CustomException.java

@@ -8,48 +8,20 @@
 public class CustomException extends RuntimeException {
     private final HttpStatus status;
     private final ErrorCode errorCode;
-    private final String detail;
 
     public CustomException(HttpStatus status, ErrorCode errorCode) {
         this.status = status;
         this.errorCode = errorCode;
-        this.detail = "";
-    }
-
-    public CustomException(HttpStatus status, ErrorCode errorCode, String detail) {
-        this.status = status;
-        this.errorCode = errorCode;
-        this.detail = detail;
-    }
-
-    public CustomException(HttpStatus status, ErrorCode errorCode, Throwable cause) {
-        this.status = status;
-        this.errorCode = errorCode;
-        this.detail = cause.getMessage();
-    }
-
-    public CustomException(HttpStatus status, CustomException customException) {
-        this.status = status;
-        this.errorCode = customException.getErrorCode();
-        this.detail = customException.getDetail();
-    }
-
-    public CustomException(HttpStatus status, Throwable cause) {
-        this.status = status;
-        this.errorCode = ErrorCode.UNKNOWN;
-        this.detail = cause.getMessage();
     }
 
     public CustomException(Exception exception) {
         if (exception.getClass() == CustomException.class) {
             CustomException customException = (CustomException) exception;
             this.status = customException.getStatus();
             this.errorCode = customException.getErrorCode();
-            this.detail = customException.getMessage();
         } else {
             this.status = HttpStatus.BAD_REQUEST;
             this.errorCode = ErrorCode.UNKNOWN;
-            this.detail = exception.getMessage();
         }
     }
 }

src/main/java/com/compono/ibackend/common/handler/CustomExceptionHandler.java

@@ -1,16 +1,23 @@
 package com.compono.ibackend.common.handler;
 
 import com.compono.ibackend.common.dto.error.response.ErrorDTO;
+import com.compono.ibackend.common.enumType.ErrorCode;
 import com.compono.ibackend.common.exception.CustomException;
+import lombok.extern.slf4j.Slf4j;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.ControllerAdvice;
 import org.springframework.web.bind.annotation.ExceptionHandler;
 
+@Slf4j
 @ControllerAdvice
 public class CustomExceptionHandler {
 
     @ExceptionHandler(CustomException.class)
     protected ResponseEntity<ErrorDTO> handleCustom400Exception(CustomException ex) {
+        ErrorCode errorCode = ex.getErrorCode();
+        log.warn(String.format("http-status={%d} code={%d} msg={%s}", ex.getStatus().value(), errorCode.getCode(),
+                errorCode.getMsg()));
+
         return ErrorDTO.toResponseEntity(ex);
     }
 }