chore: Sentry 필터링 적용 #777
chore: Sentry 필터링 적용 #777
Conversation
|
Caution Review failedThe pull request is closed. Walkthrough이번 변경 사항은 Sentry와 관련된 상수 및 예외 처리 기능을 추가하고, Sentry의 트랜잭션 전송 방식을 사용자 정의하는 새로운 클래스를 도입합니다. 또한, Sentry의 로깅 설정을 수정하여 경고 수준 이상의 이벤트만 기록하도록 변경하였습니다. 이러한 변경은 Sentry의 오류 추적 및 로깅 기능을 개선하는 데 중점을 두고 있습니다. Changes
Assessment against linked issues
Possibly related PRs
Suggested reviewers
Poem
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: 1
Outside diff range and nitpick comments (1)
src/main/java/com/gdschongik/gdsc/global/common/constant/SentryConstant.java (1)
5-5: 클래스명 수정 제안
SentryConstant클래스를SentryConstants로 변경하는 것을 고려해보세요. 여러 상수를 포함하고 있으므로 복수형을 사용하면 더 명확합니다.수정 사항 제안:
-public class SentryConstant { +public class SentryConstants {
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (4)
- src/main/java/com/gdschongik/gdsc/global/common/constant/SentryConstant.java (1 hunks)
- src/main/java/com/gdschongik/gdsc/global/config/SentryConfig.java (1 hunks)
- src/main/java/com/gdschongik/gdsc/infra/sentry/CustomBeforeSendTransactionCallback.java (1 hunks)
- src/main/resources/application-sentry.yml (1 hunks)
Additional comments not posted (2)
src/main/resources/application-sentry.yml (1)
13-13: 로그 레벨 변경 확인 필요
minimum-event-level을info에서warn으로 변경하셨습니다. 이로 인해info레벨의 이벤트는 Sentry에 전송되지 않습니다. 의도한 변경인지 확인 부탁드립니다.src/main/java/com/gdschongik/gdsc/infra/sentry/CustomBeforeSendTransactionCallback.java (1)
22-23: 특정 트랜잭션 필터링 동작 확인
KEYWORDS_TO_IGNORE에 포함된 키워드를 가진 트랜잭션을 Sentry에서 필터링하도록 구현하셨습니다. 해당 키워드를 포함하는 트랜잭션이 모니터링되지 않게 됩니다. 의도한 동작인지 확인 부탁드립니다.
| import lombok.RequiredArgsConstructor; | ||
| import org.springframework.context.annotation.Bean; | ||
| import org.springframework.context.annotation.Configuration; | ||
| import org.springframework.web.bind.MethodArgumentNotValidException; | ||
| import org.springframework.web.servlet.resource.NoResourceFoundException; |
There was a problem hiding this comment.
올바른 예외 클래스 사용 필요
NoResourceFoundException 클래스는 존재하지 않을 수 있습니다. Spring MVC에서는 존재하지 않는 엔드포인트에 대한 예외로 NoHandlerFoundException을 사용합니다. 이 클래스로 수정하는 것을 권장합니다.
수정 사항 제안:
-import org.springframework.web.servlet.resource.NoResourceFoundException;
+import org.springframework.web.servlet.NoHandlerFoundException;예외 목록에서의 변경:
-private final List<Class<? extends Throwable>> exceptionsToIgnore = List.of(
- NoResourceFoundException.class, // 존재하지 않는 엔드포인트로 요청이 들어왔을 때
+private final List<Class<? extends Throwable>> exceptionsToIgnore = List.of(
+ NoHandlerFoundException.class, // 존재하지 않는 엔드포인트로 요청이 들어왔을 때Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| import org.springframework.web.servlet.resource.NoResourceFoundException; | |
| import org.springframework.web.servlet.NoHandlerFoundException; |
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
🌱 관련 이슈
📌 작업 내용 및 특이사항
📝 참고사항
📚 기타
Summary by CodeRabbit
Summary by CodeRabbit
새로운 기능
SentryConstant클래스 추가.NoResourceFoundException및MethodArgumentNotValidException추가.CustomBeforeSendTransactionCallback클래스 추가.구성 변경
warn으로 변경하여 경고 이상의 이벤트만 기록하도록 조정.