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] MainApiGlobalExceptionHandler에 @Slf4j 어노테이션 추가 #174

Merged
merged 1 commit into from
Jan 17, 2025

Conversation

rlajm1203
Copy link
Collaborator

📌 관련 이슈

✨ PR 내용

  • @slf4j 어노테이션이 없어서 발생하는 에러 해결

🤓 리뷰어에게

@rlajm1203 rlajm1203 added the bug Something isn't working label Jan 17, 2025
@rlajm1203 rlajm1203 self-assigned this Jan 17, 2025
Copy link
Collaborator Author

@rlajm1203 rlajm1203 left a comment

Choose a reason for hiding this comment

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

Code review by ChatGPT

@@ -14,6 +15,7 @@

@RestControllerAdvice
@RequiredArgsConstructor
@Slf4j
public class MainApiGlobalExceptionHandler {
//TODO: mapper
@ExceptionHandler(ApplicationException.class)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@slf4j 어노테이션을 통해 로깅을 사용할 수 있게 되었습니다. 이로써 어떤 예외가 발생했는지 등의 정보를 로그에 남길 수 있게 되었습니다. 이외에도 예외 처리를 위해 @ExceptionHandler 어노테이션이 사용되었는데, ApplicationException이라는 예외가 발생했을 때 처리할 코드를 추가해 주셨습니다. 현재로서는 큰 문제가 없어 보입니다. 단, mapper에 대한 TODO 주석이 있어 해당 부분에 대한 구현이 필요한 것으로 보입니다. 해당 구현을 빠르게 완료하여 기능을 완성하는 것이 중요해 보입니다.

@rlajm1203 rlajm1203 merged commit a8ee2f7 into develop Jan 17, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant