Skip to content

Feature/#33 sentry#37

Merged
ImJaeOne merged 12 commits intodevfrom
Feature/#33-Sentry
Apr 6, 2025
Merged

Feature/#33 sentry#37
ImJaeOne merged 12 commits intodevfrom
Feature/#33-Sentry

Conversation

@ImJaeOne
Copy link
Copy Markdown
Collaborator

@ImJaeOne ImJaeOne commented Apr 5, 2025

💡 관련이슈

🍀 작업 요약

Sentry 기초 설정
Sentry ↔ Slack 연동
500번대 에러에 대해서만 Sentry 포착하도록 설정

💬 리뷰 요구 사항

기존에 사용하시던 fetch 대신 fetchWithSentry 사용해주시면 됩니다.

await fetchWithSentry(SIGN_UP, {
    method: POST,
    body: JSON.stringify(data),
  });

method, body를 인자로 받습니다. (GET과 DELETE 요청의 경우 body는 필요하지 않으며, 해당 경우를 고려해 body는 옵셔널로 처리되어 있습니다.)

multipart/form-data 형식일 경우 headers를 명시하지 않아야 하므로, headers는 내부에서 자동 처리되도록 구현했습니다.

💛 미리보기

스크린샷 2025-04-06 오전 12 07 21

✔️ 이슈 닫기

Ref #33

@ImJaeOne ImJaeOne self-assigned this Apr 5, 2025
@ImJaeOne ImJaeOne added ✨ Feature 기능 개발 🕶️ 재원 재원 D-1 긴급하게 처리해야 하지만 D-0보다는 상대적으로 여유 있는 작업입니다. 중요한 작업으로 빠른 진행이 요구됩니다. labels Apr 5, 2025
Copy link
Copy Markdown
Collaborator

@choichangyeon choichangyeon left a comment

Choose a reason for hiding this comment

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

리뷰 확인 후 피드백 부탁드리겠습니다!

Comment thread sentry.edge.config.ts Outdated
Comment thread sentry.server.config.ts Outdated
Comment thread src/instrumentation-client.ts Outdated
Copy link
Copy Markdown
Collaborator

@hye410 hye410 left a comment

Choose a reason for hiding this comment

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

재원님 sentry 적용까지 나서서 다 해주시고 너무 감사합니다 🙇‍♀️
그리고 덕분에 route handler에서 에러를 던질 때 왜 status 코드를 분리해야 하는지도 알게 되었어욥!!
짱👍👍👍👍👍

Copy link
Copy Markdown
Collaborator

@parkminjo parkminjo left a comment

Choose a reason for hiding this comment

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

재원님 덕분에 Sentry 로직도 금방 사용할 수 있게 됐네요! 감사합니다:)
해당 내용을 좀 보면서 공부해야겠어요

Comment thread src/instrumentation.ts
Copy link
Copy Markdown
Collaborator

@choichangyeon choichangyeon left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@ImJaeOne ImJaeOne merged commit 6bd4bad into dev Apr 6, 2025
1 check passed
@ImJaeOne ImJaeOne deleted the Feature/#33-Sentry branch April 6, 2025 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

D-1 긴급하게 처리해야 하지만 D-0보다는 상대적으로 여유 있는 작업입니다. 중요한 작업으로 빠른 진행이 요구됩니다. ✨ Feature 기능 개발 🕶️ 재원 재원

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants