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

HttpMessageNotReadable 예외 처리 #98

Merged
merged 2 commits into from
Aug 14, 2024

Conversation

drbug2000
Copy link
Collaborator

📝 요약

  • 이슈 번호 : Feat : HttpMessageNotReadableException 처리 #97
    request JSON 형식에 문제가 생겼을 경우 발생하는 HttpMessageNotReadableException 처리를 추가했습니다.
    해당 예외 발생시 error 코드와 request body에 무엇이 담겨 있었는지 함께 응답합니다.

🔖 변경 사항

  • RequestCaching Fliter를 사용해서 Request body의 정보를 캐싱합니다.
  • HttpNotReadableException 함수를 추가해 에러를 처리합니다.

📸 확인 방법 (선택)

image


📌 PR 진행 시 이러한 점들을 참고해 주세요

* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

@drbug2000 drbug2000 linked an issue Aug 14, 2024 that may be closed by this pull request
Copy link
Collaborator

@seongjunnoh seongjunnoh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@drbug2000 drbug2000 merged commit c4bda32 into develop Aug 14, 2024
3 checks passed
@hyunn522 hyunn522 deleted the feat/#97httpmessagenotreadableexception branch August 14, 2024 14:41
seongjunnoh pushed a commit that referenced this pull request Oct 30, 2024
…exception

HttpMessageNotReadable 예외 처리
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.

Feat : HttpMessageNotReadableException 처리
2 participants