Skip to content

Conversation

@wocjf0513
Copy link
Contributor

😁 Issue Link

closes #374

😆 To Reviewers

사용하기 위해

  1. gradle.properties라는 파일이 필요합니다 (build.gradle에서 환경변수 사용이 필요해서)
  2. application.yml에도 sentry설정 추가 필요
sentry:  
  dsn: https://66894e10453eefc0822f7215608b0a33@o4507135273205760.ingest.us.sentry.io/4507135275171841  

해당 코드가 필요해요!

😚 Reference

https://docs.sentry.io/platforms/java/guides/spring-boot/

📌 PR 진행 시 이러한 점들을 참고해 주세요

  • Reviewer 분들은 코드 리뷰 시 좋은 코드의 방향을 제시하되, 코드 수정을 강제하지 말아 주세요.
  • Reviewer 분들은 좋은 코드를 발견한 경우, 칭찬과 격려를 아끼지 말아 주세요.
  • Review는 특수한 케이스가 아니면 Reviewer로 지정된 시점 기준으로 3일 이내에 진행해 주세요.
  • Comment 작성 시 Prefix로 P1, P2, P3 를 적어 주시면 Assignee가 보다 명확하게 Comment에 대해 대응할 수 있어요
    • P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
    • P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
    • P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

@wocjf0513 wocjf0513 added the 🛠feature 기능 구현 label Apr 23, 2024
@wocjf0513 wocjf0513 self-assigned this Apr 23, 2024
@wocjf0513 wocjf0513 closed this Apr 23, 2024
@wocjf0513 wocjf0513 reopened this Apr 23, 2024
@github-actions
Copy link

github-actions bot commented Apr 23, 2024

Test Results

95 tests  ±0   95 ✅ ±0   14s ⏱️ ±0s
58 suites ±0    0 💤 ±0 
58 files   ±0    0 ❌ ±0 

Results for commit c1f8c6c. ± Comparison against base commit 862ae20.

♻️ This comment has been updated with latest results.

@wocjf0513 wocjf0513 force-pushed the feature/#374_sentry_for_monitoring_server branch from 7d5f82a to 133513b Compare April 23, 2024 12:12
Copy link
Contributor

@Aleexender Aleexender left a comment

Choose a reason for hiding this comment

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

굿굿 CI 만 손보면 될듯!? 내일 알려주라...


@ExceptionHandler
public ResponseEntity<ResponseDTO<String>> bindException(BindException e) {
Sentry.captureException(e);
Copy link
Contributor

Choose a reason for hiding this comment

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

오 에러로그볼려구 했구낭 굿굿! 나중에 알려줘

Copy link
Contributor Author

Choose a reason for hiding this comment

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

예스~~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🛠feature 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants