chore: log level info 로 변경 및 stack trace 제거 #838
Conversation
Walkthrough글로벌 예외 처리 핸들러에서 로깅 수준을 Changes
Assessment against linked issues
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
src/main/java/com/gdschongik/gdsc/global/exception/GlobalExceptionHandler.java (1)
40-40: 검증 오류 메시지 개선 제안검증 오류를 info 레벨로 변경한 것은 적절하나, 디버깅을 위해 더 자세한 정보를 로그에 포함하는 것이 좋을 것 같습니다.
다음과 같이 필드명과 오류 내용을 함께 로깅하는 것을 제안드립니다:
- log.info("METHOD_ARGUMENT_NOT_VALID : {}", e.getMessage()); + log.info("METHOD_ARGUMENT_NOT_VALID - Field: {}, Error: {}", + e.getBindingResult().getFieldError().getField(), + e.getBindingResult().getFieldError().getDefaultMessage());
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/main/java/com/gdschongik/gdsc/global/exception/GlobalExceptionHandler.java(1 hunks)
🔇 Additional comments (3)
src/main/java/com/gdschongik/gdsc/global/exception/GlobalExceptionHandler.java (3)
20-20: 로그 레벨 변경이 적절합니다CustomException은 예측 가능한 비즈니스 예외이므로 info 레벨로 로깅하는 것이 적절합니다.
26-26: 결제 관련 예외 처리 확인 필요결제 관련 예외를 info 레벨로 변경한 것이 적절해 보이나, 중요한 결제 실패 케이스를 놓치지 않도록 Sentry에서 모니터링이 잘 되고 있는지 확인해주세요.
32-32: 내부 서버 오류는 error 레벨 유지가 적절합니다예측하지 못한 내부 서버 오류의 경우 error 레벨을 유지한 것이 적절합니다.
🌱 관련 이슈
📌 작업 내용 및 특이사항
📝 참고사항
예상가능한 에러에 대해서는 info 레벨로 로깅하는 것이 일반적이라고 하여 로깅 레벨을 변경하고 센트리는 기존대로 erorr 레벨에 대해 알림을 받도록 하였습니다. internal server error 에 대해서만 error 레벨을 유지했는데, 여기에서도 스택 트레이스를 제거하면 정보를 센트리에서 확인하지 못할까 싶어서 찾아보고 테스트해봤는데, stack trace 에 대한 출력 없이도 sentry 에서 stack trace 조회가 가능하여 error 레벨에서도 스택 트레이스 출력을 제거 했습니다.
📚 기타
Summary by CodeRabbit