Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

부산대 BE_정재빈 4주차 과제(2단계) #344

Open
wants to merge 13 commits into
base: jaebin2019
Choose a base branch
from

Conversation

JaeBin2019
Copy link

@JaeBin2019 JaeBin2019 commented Jul 19, 2024

  1. Response 값에서 직접 VO 객체로 데이터를 검증하면, Custom Excpetion 을 HttpMessageNotReadableException 로 Wrapping 해서 GlobalExcepionHandler 에 처리하는 문제를 발견하여, 여러 방법을 시도하다 각 Controller에서 해당 에러를 처리하는 방식으로 수정하였습니다
    // GlobalException Handler 에서 처리할 경우,
    // RequestBody에서 발생한 에러가 HttpMessageNotReadableException 로 Wrapping 이 되는 문제가 발생한다
    // 때문에, 해당 에러로 Wrapping 되기 전 Controller 에서 Domain Error 를 처리해주었다
    @ExceptionHandler(DomainValidationException.class)
    public ResponseEntity<ErrorResponseDto> handleOptionValidException(DomainValidationException e) {
        System.out.println(e);
        return ResponseHelper.createErrorResponse(e.getErrorCode());
    }

늦어서 죄송합니다.

추가적인 ResponeBody 관련과 Service 테스트는 차후 여유가 나면 처리하도록 하겠습니다. 늦어서 죄송합니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant