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

fix(#105): 에러 메시지 수정 #121

Merged
merged 1 commit into from
Sep 7, 2024
Merged

fix(#105): 에러 메시지 수정 #121

merged 1 commit into from
Sep 7, 2024

Conversation

cabbage16
Copy link
Member

  • Request Body의 값을 잘못 넣었을 때 지정한 에러 메시지 대신 내부 에러 메시지 전체가 출력되는 오류를 해결했어요.

🎫 관련 이슈

close #105


📄 개요

Request Body에 잘못된 값을 넣었을 때 서버의 에러 메시지가 그대로 출력되는 것을 해결했습니다.


🔨 작업 내용

  • MethodArgumentNotValidException이 발생했을 때 서버 에러메시지 대신 설정한 에러 메시지가 응답으로 나오도록 변경했습니다.

🏁 확인 사항

  • 테스트를 완료했나요?
  • API 문서를 작성했나요?
  • 코드 컨벤션을 준수했나요?
  • 불필요한 로그, 주석, import 등을 삭제했나요?

🙋🏻 덧붙일 말

Enum 유효성을 검증하기 위해서는 @JsonCreator를 사용해서 파라미터로 해당 enum 이외의 값이 전달되면 Validator로 null을 전달해서 오류를 발생시켜야 하는데, 상황에 따라서 enum 값에 null을 허용하는 경우(원서 전체 조회시 FormType)도 있고 아닌 경우(원서 제출시 FormType)도 있어서 이에 대한 처리가 어려워 포기했습니다.

대신에 서버 에러메시지가 그대로 응답에 포함되는 것은 해결했습니다.

- Request Body의 값을 잘못 넣었을 때 지정한 에러 메시지 대신 내부 에러 메시지 전체가 출력되는 오류를 해결했어요.
@cabbage16 cabbage16 requested a review from a team as a code owner September 7, 2024 09:08
Copy link

github-actions bot commented Sep 7, 2024

이렇게 리뷰해주세요.

🖐️ 규칙

  1. 개선이 필요한지 충분히 설명해주세요.
  2. 답을 알려주기보다는 스스로 고민하고 개선 방법을 선택할 수 있게 해주세요.
  3. 코드를 깔끔하게 유지하고, 일관되게 구현하도록 안내해 주세요.
  4. 리뷰 과정이 숙제검사가 아닌 학습과정으로 느낄 수 있게 리뷰해 주세요.
  5. 리뷰를 위한 리뷰를 하지 마세요. 피드백 할 게 없으면 칭찬해 주세요.

🥬 우선순위

리뷰 중요도에 따라 다음과 같은 prefix를 붙여주세요.

예시 - P5) 쓸모없는 주석 삭제해주세요.

  • P1: 꼭 반영해주세요 (Request changes)
  • P2: 적극적으로 고려해주세요 (Request changes)
  • P3: 웬만하면 반영해 주세요 (Comment)
  • P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
  • P5: 그냥 사소한 의견입니다 (Approve)

Copy link

sonarcloud bot commented Sep 7, 2024

@jyj1289 jyj1289 merged commit 173063a into develop Sep 7, 2024
3 checks passed
@jyj1289 jyj1289 deleted the perf/#105 branch September 7, 2024 09:20
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.

[개선] Enum 유효성 검증 어노테이션 및 에러 메시지 노출 방지
2 participants