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.

Sign up for GitHub

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

Jump to bottom

[#414] 실시간 에러감지를 위한 메신져 알림 로직 구현 #415

Merged
merged 6 commits into from
Dec 11, 2023
Merged

[#414] 실시간 에러감지를 위한 메신져 알림 로직 구현 #415

merged 6 commits into from
Dec 11, 2023

Conversation

angelSuho
Copy link
Member

Issue

Issue 내용

  • DiscordService, SlackService 클래스를 추가하여 각 SNS로 실시간으로 알림을 받을 수 있는 로직을 구현하였습니다.
  • 전역예외핸들러 클래스에 예외를 캐치했을 때, 메신져에도 알릴수 있도록 추가하였습니다.
  • 인증관련 예외는 전역예외핸들러에서 잡을 수 없어, 직접 인증 로직 중, 예외가 발생한 구역에서 예외를 알릴 수 있도록 추가하였습니다.

@angelSuho angelSuho added 🚀 Feature New feature or request 🛠 Setting labels Dec 10, 2023
@angelSuho angelSuho self-assigned this Dec 10, 2023
@angelSuho angelSuho linked an issue Dec 10, 2023 that may be closed by this pull request
실시간 오류감지를 위한 sns 로직 적용 #414
Closed
gkfktkrh153
gkfktkrh153 approved these changes Dec 10, 2023
View reviewed changes
Copy link
Member

@gkfktkrh153 gkfktkrh153 left a comment

Choose a reason for hiding this comment

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

DiscordUtil이라는게 결국 DiscordMessageObject인거 같은데 Util로 만든 이유가 있을까여??
코드 흐름이 잘 이해가 안가서 한번 설명해주시면 좋을거같습니다~
일단 머지는 안할게요!

src/main/java/com/festival/common/security/JwtAuthenticationFilter.java
ErrorCode errorCode = e.getErrorCode();

// slackService.sendSlackAlertLog(e, request);
Copy link
Member

Choose a reason for hiding this comment

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

이거는 일부러 비활성화 시킨건가요??

Copy link
Member Author

Choose a reason for hiding this comment

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

DiscordUtil에서 오브젝트 생성과 http 요청을 만드는 로직이 같이 포함되어 있어서 Util로 생성하였습니다

Copy link
Member Author

Choose a reason for hiding this comment

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

slack도 포함되어있으면 Discord와 Slack 둘다 알림이 가게되어서 우선은 Discord로 알림을 받자고 얘기가 되어 Discord만 알림을 받을 수 있도록 Slack을 주석처리 해두었습니다

Copy link
Member

Choose a reason for hiding this comment

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

그러면 DisocdObject를 따로 가져가고 http요청을 만드는 로직을 유틸로 분리하는게 더 좋아보이긴 합니다

Copy link
Member Author

Choose a reason for hiding this comment

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

Slack과 다르게 Discord는 api가 만들어져 있지 않아서 http 요청을 만들어서 전송해야합니다. 이때 중요한 것이 Discord에서 제시한 파라미터의 명을 꼭 지켜야 알림을 받을 수 있는데요. 따라서 각 json 오브젝트를 매번 작성할 것 같아 Util 클래스 처럼 오브젝트를 생성해서 파라미터를 관리한다고 알고 있습니다.
그래서 전체 과정을 보자면,
Discord에 맞는 Json 생성 -> Json을 http 요청으로 Discord 채널에 전송 -> Discord 채널에 알림 발생

인 것 같습니다

Abbbyy
Abbbyy approved these changes Dec 11, 2023
View reviewed changes
Copy link
Collaborator

@Abbbyy Abbbyy left a comment

Choose a reason for hiding this comment

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

slack, discord 에러 감지 시 메신저 알림 서비스 구현 확인하였습니다!

@angelSuho angelSuho merged commit 1a09859 into development Dec 11, 2023
@angelSuho angelSuho deleted the feature/414_SnsErrorAlert branch December 11, 2023 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gkfktkrh153 gkfktkrh153 gkfktkrh153 approved these changes

@Abbbyy Abbbyy Abbbyy approved these changes

Assignees

@angelSuho angelSuho

Labels
🚀 Feature New feature or request 🛠 Setting
Projects
millicon
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

실시간 오류감지를 위한 sns 로직 적용
3 participants
@angelSuho @gkfktkrh153 @Abbbyy