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

[FEAT] Global Handler 및 Internal Server Error 슬랙 알림 추가 #188

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

GaHee99
Copy link
Contributor

@GaHee99 GaHee99 commented Jun 9, 2024

Related Issue 🚀

Work Description ✏️

  • RuntimeException과, 슬랙 알림 Exception을 잡습니다.
  • 클라이언트가 받을 response규격을 고려해서, 기존과 500에러 response틀을 맞추고자 합니다.
스크린샷 2024-06-09 오후 6 52 07
-> slack알림은 에러 메시지만 받습니다!
스크린샷 2024-06-09 오후 6 52 28

PR Point 📸

  • 에러 처리에 대한 로그도 잘 심어놔야 할 것 같아요,
  • 현재 config와 common폴더에 추가해놨는데, 폴더 위치를 옮기거나 수정해야할 것 같아요! 좋은 의견 있으시다면 추천해주세욤~
  • 500에러 처리에 추가적으로 필요한 데이터가 있다면 꼭 말씀해주세요!

close #187

@GaHee99 GaHee99 requested review from dragontaek-lee and kjy-asl June 9, 2024 09:53
@GaHee99 GaHee99 self-assigned this Jun 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] Error Handler적용
2 participants