Feat : GlobalException 세팅

[yarogono]

Checklist

• [x] 테스트 코드를 작성하셨나요?
• [x] Exception 처리를 꼼꼼하게 하셨나요?
• [x] 머지하기 전에 팀원과 크로스 체크가 되었나요?

작업내용

• Global Exception Handler 초기 세팅
• Exception Code를 Enum 클래스를 사용해서 서버와 클라이언트 분리
• Runtime Exception들을 Global Exception Handler 처리를 통해 동일한 양식의 ResponseEntity 객체로 응답
• ExceptionResponseDto를 통해 ResponseEntity 외에도 추가적으로 필요한 정보들을 Body에 담아서 응답

Questions

💬 질문 사항이에요!

• ServerExceptionCode에 code 부분을 500 Inter Server Error 라고 적기는 했는데, 이 부분에 대해서 고민을 해도 어떤게 더 효율적인지 몰라서 아직은 정하지 못했습니다. Exception code를 프로젝트에 맞춰서 정하고 문서화를 해서 클라이언트 개발자에게 공유하는게 좀 더 좋지 않을까 라는 고민도 했는데 아직 갈피를 못 잡아서 질문 남깁니다.

🤷‍♂ ️확인 받고 싶은 부분이에요!

• ServerExceptionCode.java, GlobalExceptionHandler.java 두 개가 주로 코드를 작성한 부분입니다.

[f-lab-elkein]

질문에 대해 먼저 답변을 드리자면 HTTP Status Code는 Frontend가 팝업으로 오류 처리만해도 되는가 501처럼 인증 화면이나 로직으로 다시 보내야되는가 등에 따라서 합의를 보면 되는 부분입니다.

보통 코드 정의된 부분을 공유하는 수준으로 처리하는 경우가 많지만 예외 정보를 REST API로 제공하는 것도 좋습니다. 런타임 동기화가 되고 죽은 문서를 만들지않기 때문입니다.

[f-lab-elkein]

코드는 전반적으로 잘 처리하셨습니다. 실제 비즈니스 로직을 다룰 때 어떤지 보면서 같이 피드백 드릴 부분이 있을 수도 있을 거 같아요

[yarogono]

질문에 대해 먼저 답변을 드리자면 HTTP Status Code는 Frontend가 팝업으로 오류 처리만해도 되는가 501처럼 인증 화면이나 로직으로 다시 보내야되는가 등에 따라서 합의를 보면 되는 부분입니다.

보통 코드 정의된 부분을 공유하는 수준으로 처리하는 경우가 많지만 예외 정보를 REST API로 제공하는 것도 좋습니다. 런타임 동기화가 되고 죽은 문서를 만들지않기 때문입니다.

감사합니다 멘토님 생각해보니 따로 에러코드를 만들면 문서를 만들어야하는 소요가 생기고, 그 문서의 에러코드들이 레거시화 되어서 아무도 안보면 문제가 생기겠네요. 전 회사에서 증권가 파견 갔을 때도 에러 코드를 'E1101'와 같은 약속된 코드를 썼었는데, 그 문서가 제대로 인수인계가 안되어서 아무도 에러코드를 모르는 문제가 떠올랐네요. 감사합니다!

[jeongaaa]

👍