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

슬랙 알림 로직 분리 및 유동적 알림 설정 가능하도록 변경 #385

Closed
8 tasks done
limehee opened this issue Jun 19, 2024 · 0 comments · Fixed by #386
Closed
8 tasks done

슬랙 알림 로직 분리 및 유동적 알림 설정 가능하도록 변경 #385

limehee opened this issue Jun 19, 2024 · 0 comments · Fixed by #386
Assignees
Labels
🔨 Refactor 코드 수정 및 개선

Comments

@limehee
Copy link
Collaborator

limehee commented Jun 19, 2024

Describe

슬랙 알림 로직을 도메인 서비스로부터 분리하고, 알림 설정을 DB에서 관리하여 재배포 없이 알림을 ON/OFF 할 수 있도록 개선합니다.

Tasks

  • 알림 엔티티 정의

  • NotificationSetting 엔티티를 생성하여 알림 이벤트 타입(AlertType), 활성화 상태, Webhook URL 등을 관리합니다.

  • 엔티티는 알림 설정을 저장하기 위해 필요한 필드를 포함합니다.

  • 알림 레포지토리 생성

  • NotificationSettingRepository를 생성하여 알림 설정 엔티티를 CRUD 방식으로 관리합니다.

  • 특정 이벤트 타입에 대한 설정을 검색하는 기능을 포함합니다.

  • 알림 서비스 구현

  • NotificationSettingService를 생성하여 DB의 설정을 기반으로 슬랙 알림을 전송하는 로직을 담당합니다.

  • 이벤트 타입을 받아서 해당 이벤트의 설정이 활성화된 경우 슬랙 알림을 전송합니다.

  • SlackServiceHelper를 통해 실제 알림을 전송합니다.

  • SlackService 수정

  • SlackService를 수정하여 지정된 Webhook URL로 알림을 전송할 수 있도록 메서드를 수정합니다.

  • 기존 메서드를 새로 구현된 알림 서비스와 연동합니다.

  • 도메인 서비스 수정

  • 도메인 서비스들이 직접 SlackService를 사용하지 않고, NotificationSettingService를 사용하여 알림을 전송하도록 수정합니다.

  • 예: RedisIpAccessMonitorService가 NotificationSettingService를 통해 알림을 전송하도록 변경합니다.

  • DB 설정 연동

  • 알림 설정을 초기화하거나 변경할 수 있는 방법을 제공하여 애플리케이션 재배포 없이 설정을 변경할 수 있도록 합니다.

  • 예: DB에 알림 설정 데이터를 미리 삽입하거나 관리 도구를 통해 변경할 수 있도록 구현합니다.

  • 슬랙 알림 조회 API 구현

  • 특정 알림 이벤트 타입(AlertType)에 대한 현재 알림 설정을 조회할 수 있는 API를 추가합니다.

  • 슬랙 알림 설정 변경 API 구현

  • 알림 이벤트 타입(AlertType)과 활성화 상태를 변경할 수 있는 API를 추가합니다.

ETC

이번 작업에서는 발행-구독 패턴(Publisher-Subscriber Pattern)을 사용하여 알림 로직을 개선합니다. 발행-구독 패턴은 이벤트 발생 시 이를 여러 서브스크라이버가 구독하고 처리하는 구조로, 시스템 간의 결합도를 낮추고 확장성을 높입니다. 이를 통해 알림 설정을 DB에서 관리할 수 있어, 알림의 활성화/비활성화를 애플리케이션 재배포 없이 동적으로 조정할 수 있습니다.

@limehee limehee added the 🔨 Refactor 코드 수정 및 개선 label Jun 19, 2024
@limehee limehee self-assigned this Jun 19, 2024
@limehee limehee changed the title 슬랙 메시지 개선 슬랙 알림 로직 분리 및 유동적 알림 설정 가능하도록 변경 Jun 19, 2024
@limehee limehee closed this as completed Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 Refactor 코드 수정 및 개선
Projects
None yet
1 participant