-
Notifications
You must be signed in to change notification settings - Fork 0
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
[feat] : 에러 처리를 추가하며, 메세지를 정제하여 전달한다 #24
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
정말 수고 많았습니다!!
이 에러 처리들은 나중에도 두고두고 쓸 수 있을 거 같아요!!
logError("MissingServletRequestParameterException", errorMessage); | ||
return ApiResponse.onFailure(ErrorStatus._BAD_REQUEST, errorMessage); | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 필수 파라미터 여러개 값이 없으면 여러개 다에 대해서 에러메시지 난오는 거죠???
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거는 파라미터 1개마다 각각 에러가 발생해서 1개만 처리가 되더라구요 ㅠ
가능하면 다음에 여러개를 동시에 처리할 수 있는 방법을 찾아봐도 좋을 것 같네요!
아 추가로 RequestBody에서도 Valid말고 Validated 써도 돼요!! 주요 차이점 언제 사용해야 하나? |
오호 굿굿 그러면 클래스단에 |
이건 안되네요 ㅎㅎ 제 생각에는 기존처럼 RequestBody에는 |
✅ PR 유형
어떤 변경 사항이 있었나요?
📝 작업 내용
이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)
@PathParam
->@RequestParam
으로 변경@Validated
어노테이션을 붙이고, 파라미터에@NotBlank
,@Min(1)
어노테이션을 붙이며 파라미터에 대한 유효성 검증을 추가하였습니다.@Validated
어노테이션을 붙이지 않는다면 파라미터에 대한 유효성 검증이 제대로 되지 않습니다. (RequestBody에는@Valid
만 붙이면 가능합니다)ConstraintViolationException 처리 (쿼리 파라미터에 올바른 값이 들어오지 않은 경우)
MissingServletRequestParameterException 처리 (필수 쿼리 파라미터가 입력되지 않은 경우)
MethodArgumentNotValidException
처리 추가@Override
를 통해서 에러 처리를 추가했습니다.MethodArgumentNotValidException 처리 (RequestBody로 들어온 필드들의 유효성 검증에 실패한 경우)
그 외 에러 처리 추가
HttpRequestMethodNotSupportedException 처리 (지원하지 않는 HTTP 메소드 요청이 들어온 경우)
내부 서버 에러 처리 (500) 응답 변경
✏️ 관련 이슈
본인이 작업한 내용이 어떤 Issue Number와 관련이 있는지만 작성해주세요
🎸 기타 사항 or 추가 코멘트
하면서 에러 처리에 대한 강제 학습이 된 것 같습니다...
저도 나중에 다시 볼 겸 도움 됐던 블로그 글들 정리해둡니다!