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

dev 서버 Sentry 설정 #182

Merged
merged 13 commits into from
Mar 17, 2024
Merged

dev 서버 Sentry 설정 #182

merged 13 commits into from
Mar 17, 2024

Conversation

Sehee-Lee-01
Copy link
Member

@Sehee-Lee-01 Sehee-Lee-01 commented Mar 16, 2024

🎫 관련 이슈

Resolves #175

✅ 구현 내용

dev 서버 Sentry 설정

  • 기존 warn 레벨 이상 예외들은 로그와 함께 Sentry로 알림 보내도록 설정
  • sentry-jdbc 의존성 추가로 SQL 쿼리도 인터셉트하여 sentry에서 분석할 수 있도록 설정
  • 위 설정을 위한 sidepeek_backend_sacret dev 변경

💬 코멘트

  • 블로그 기록: [Spring] 서버에 Sentry를 적용해보자
  • sentry가 여러명으로 팀 구성하면 유료 요금제를 내야하더라구요... 그래서 아래 두 가지 방안을 생각해보았습니다! 둘 다 저는 상관 없으니 편하게 이야기 해주세요!
    • 무료 요금제로 한 명이 대표로 Sentry 관리하기
    • 유료 요금제 체험판으로 2주동안 다같이 자유롭게 사용하기. 단 2주(약 13일..?)가 지나면 다시 개인 관리ㅠㅠ
  • 참고로 정말 야속하게도 sentry로 직접 슬랙 연동해서 알림 보내는 것은 유료더라구요...ㅠㅠ 그래서 서버에 직접 슬랙을 연동하는 방법밖에 없어서 이 부분은 이어서 제가 찾아보도록 하겠습니다! 😭

@Sehee-Lee-01 Sehee-Lee-01 added the ⚙️ 세팅했슈 환경 세팅 label Mar 16, 2024
@Sehee-Lee-01 Sehee-Lee-01 self-assigned this Mar 16, 2024
Copy link

📝 Jacoco Test Coverage

Overall Project 70.71% -0.54%
Files changed 0%

File Coverage
MediaExceptionHandler.java 13.21% -39.62%
GlobalExceptionHandler.java 3.1% -7.96%

Copy link
Contributor

@yenzip yenzip left a comment

Choose a reason for hiding this comment

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

Sentry 적용 정말 해보고 싶었는데 적용해주셔서 감사합니다..!
블로그 글도 잘 봤어요!
고민과 공부한 흔적이 많이 드러나는 글이었습니다.
고생 많으셨습니다 ❤️‍🔥

@Sehee-Lee-01 Sehee-Lee-01 merged commit c760748 into dev Mar 17, 2024
2 checks passed
@Sehee-Lee-01 Sehee-Lee-01 deleted the chore/#175-connect-sentry branch March 17, 2024 07:51
Copy link

sentry-io bot commented Mar 18, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ InvalidAuthenticationException: 유효하지 않은 토큰입니다. POST /api/v1/auth/reissue View Issue
  • ‼️ HttpRequestMethodNotSupportedException: Request method 'POST' is not supported POST /api/v1/projects/files View Issue
  • ‼️ NullPointerException: Cannot invoke "Object.getClass()" because "constant" is null GET /api/v1/projects View Issue
  • ‼️ AccessDeniedException: 작성자 Id가 로그인한 회원 Id와 일치하지 않습니다. POST /api/v1/projects View Issue
  • ‼️ BadCredentialsException: 비밀번호가 일치하지 않습니다. POST /api/v1/auth/login View Issue

Did you find this useful? React with a 👍 or 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙️ 세팅했슈 환경 세팅
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants