[Release] V2.1.2 로그 관련 기능 수정 #308
Conversation
Co-authored-by: 나용준 <[email protected]>
* fix: security context에서 꺼내오도록 수정 * fix: gemini 수정 사항 적용 --------- Co-authored-by: 나용준 <[email protected]>
Co-authored-by: 나용준 <[email protected]>
Summary of ChangesHello @yongjun0511, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 Pull Request는 V2.1.2 릴리즈를 위한 로그 관련 기능 수정 및 데이터 업데이트를 포함합니다. 주요 변경 사항으로는 DODAM 식당의 가격 조정, Slack을 통한 서버 에러 로그 메시지 형식 개선, 그리고 컨트롤러 로그에서 사용자 ID를 가져오는 로직의 리팩토링이 있습니다. 이러한 변경을 통해 시스템의 안정성과 로그의 가독성이 향상될 것으로 기대됩니다. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. 코드 흐름 따라 로그는 길을 밝히네 버그는 숨지 못해 Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이번 PR은 로깅 관련 기능을 개선하는 데 중점을 두었습니다. 특히 ControllerLogAspect에서 SecurityContextHolder를 사용해 사용자 ID를 가져오도록 변경한 점은 매우 훌륭한 리팩토링입니다. 기존 방식보다 훨씬 안정적이고 표준적인 방법입니다. 또한, 식당 가격 정보와 슬랙 에러 메시지 형식을 수정한 내용도 확인했습니다. 전반적으로 좋은 변경이라고 생각합니다. ControllerLogAspect.java 파일의 가독성을 높이기 위한 작은 제안을 하나 추가했습니다.
| if (authentication != null && authentication.isAuthenticated() && authentication.getPrincipal() instanceof CustomUserDetails) { | ||
| CustomUserDetails userDetails = (CustomUserDetails) authentication.getPrincipal(); | ||
| return String.valueOf(userDetails.getId()); | ||
| } |
There was a problem hiding this comment.
Java 16 이상을 사용하고 있다면, instanceof 패턴 매칭을 활용하여 코드를 더 간결하게 만들 수 있습니다. 이렇게 하면 타입 확인과 변수 할당을 한 줄로 처리할 수 있어 가독성이 향상됩니다.
if (authentication != null && authentication.isAuthenticated() && authentication.getPrincipal() instanceof CustomUserDetails userDetails) {
return String.valueOf(userDetails.getId());
}
No description provided.