Skip to content

[공통] Sentry 클라이언트 노이즈 에러 필터링 (#1222)#1223

Merged
ff1451 merged 1 commit intodevelopfrom
chore/#1222/sentry-noise-filter
Apr 1, 2026
Merged

[공통] Sentry 클라이언트 노이즈 에러 필터링 (#1222)#1223
ff1451 merged 1 commit intodevelopfrom
chore/#1222/sentry-noise-filter

Conversation

@ff1451
Copy link
Copy Markdown
Contributor

@ff1451 ff1451 commented Mar 31, 2026

What is this PR? 🔍

Changes 📝

  • src/instrumentation-client.tsbeforeSend 훅을 추가하여 클라이언트에서 조치 불가능한 노이즈 에러를 필터링
    • Axios: ERR_NETWORK, ECONNABORTED, ERR_CANCELED
    • fetch/Safari: Load failed, Failed to fetch
    • Next.js: ChunkLoadError

ScreenShot 📷

N/A

Test CheckList ✅

  • 프로덕션 배포 후 Sentry에서 해당 노이즈 에러가 더 이상 수집되지 않는지 확인
  • 실제 서버 에러(4xx, 5xx)는 정상적으로 수집되는지 확인

Precaution

✔️ Please check if the PR fulfills these requirements

  • It's submitted to the correct branch, not the develop branch unconditionally?
  • If on a hotfix branch, ensure it targets main?
  • There are no warning message when you run yarn lint

Summary by CodeRabbit

릴리스 노트

  • 버그 수정
    • 네트워크 타임아웃, 연결 오류, 요청 취소, 페칭 실패, 청크 로드 오류 등 특정 유형의 오류 보고를 자동으로 필터링하여 오류 추적 시스템의 노이즈를 줄였습니다.

@github-actions github-actions Bot requested review from dooohun and hyejun0228 March 31, 2026 08:47
@ff1451 ff1451 self-assigned this Mar 31, 2026
@ff1451 ff1451 added the ⚙ Setting 개발 환경 세팅 label Mar 31, 2026
클라이언트에서 조치 불가능한 네트워크 에러가 Sentry에 보고되는 것을 방지하기 위해
beforeSend 훅으로 노이즈 에러를 필터링합니다.

- Axios: ERR_NETWORK, ECONNABORTED, ERR_CANCELED
- fetch/Safari: Load failed, Failed to fetch
- Next.js: ChunkLoadError

closes #1222
@ff1451 ff1451 force-pushed the chore/#1222/sentry-noise-filter branch from 8ea186e to c0d7dc2 Compare March 31, 2026 08:48
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 31, 2026

개요

src/instrumentation-client.tsSentry.init 설정에 beforeSend 훅을 추가하여 클라이언트 네트워크 오류와 배포 후 청크 로드 실패 등 필터링 불가능한 에러들을 조건부로 억제합니다.

변경 사항

Cohort / File(s) 요약
Sentry 에러 필터링
src/instrumentation-client.ts
beforeSend 훅을 추가하여 Axios 네트워크 오류(ERR_NETWORK, ECONNABORTED, ERR_CANCELED), 브라우저 fetch 실패(Load failed, Failed to fetch), ChunkLoadError를 필터링하도록 구성

코드 리뷰 소요 시간

🎯 2 (Simple) | ⏱️ ~8분

관련 PR

  • [공통] Sentry 연결 #1199: src/instrumentation-client.ts에서 Sentry 초기화 및 계측 클라이언트를 최초로 추가한 PR과 동일 파일 수정으로 관련성이 높습니다.

추천 리뷰어

  • ParkSungju01
  • dooohun
  • hyejun0228
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경의 주요 목적을 명확하게 설명합니다. Sentry 클라이언트에서 노이즈 에러를 필터링하는 것이 핵심 변경 사항입니다.
Linked Issues check ✅ Passed PR의 모든 변경 사항이 이슈 #1222의 요구사항을 충족합니다: Axios 에러(ERR_NETWORK, ECONNABORTED, ERR_CANCELED), fetch/Safari 에러(Load failed, Failed to fetch), Next.js ChunkLoadError 필터링이 구현되었습니다.
Out of Scope Changes check ✅ Passed 모든 변경 사항이 이슈 #1222의 범위 내에 있습니다. Sentry beforeSend 훅 추가만 포함되어 있으며, 다른 불필요한 변경은 없습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/#1222/sentry-noise-filter

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@dh2906 dh2906 left a comment

Choose a reason for hiding this comment

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

냠냠

@ff1451 ff1451 merged commit 0196787 into develop Apr 1, 2026
3 checks passed
@github-actions github-actions Bot deleted the chore/#1222/sentry-noise-filter branch April 1, 2026 03:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚙ Setting 개발 환경 세팅

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[공통] Sentry 클라이언트 노이즈 에러 필터링

2 participants