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

[FEAT] AWS SQS를 이용한 푸시알림 구현 #91

Merged
merged 8 commits into from
Aug 12, 2023

Conversation

jun02160
Copy link
Member

@jun02160 jun02160 commented Aug 12, 2023

📌 관련 이슈

close #87

✨ 어떤 이유로 변경된 내용인지

  • Message 관련 ENUM 추가

    • SQS 대기열로 보낼 때 헤더명을 지정해서 보내는 방식
  • HttpServletRequest 와 같은 웹 요청 객체 자체가 대기열에 들어가지 못해, 기존의 Slack으로 에러메시지를 보내는 부분에서 HttpServletRequest 타입을 requestMethod(String), requestURI(String)으로 나누어서 보냄

  • FCM 전송 시, 유저 식별은 항상 유저 테이블에 저장된 고유의 FCM TOKEN만을 이용한다. ← 기존에 userId를 넘겨줬던 코드 개선

  • NotificationService 를 API 서버에 두고 푸시알림을 보낼 이벤트에 적용

    **정리하자면, API서버(Service)에서 SQS 대기열에 추가(produce) → 대기열에 있는 알림을 푸시서버에서 꺼내서(consume) 파이어베이스로 전송

    • 부모자식이 서로 답변을 완료했을 때의 처리를 위와 같이 할 수 있다!
    • QnAService 에서 기존의 fcmService.pushOpponentReply()notificationService.pushOpponentReply() 로 변경

🙏 검토 혹은 리뷰어에게 남기고 싶은 말

  • 스케줄링 작업은 아직 SQS로 구현되지 않은 상태입니다. 더 알아보고 구현할 수 있다면 도전해보겠습니다!
  • 노션 Secrets에 SQS 관련 환경변수 추가해뒀습니다

@jun02160 jun02160 added 🔥 Pull Request PR 날림 예준🍒 ✨Feat 새로운 기능 추가 labels Aug 12, 2023
@jun02160 jun02160 requested a review from ddongseop August 12, 2023 14:25
@jun02160 jun02160 self-assigned this Aug 12, 2023
@jun02160 jun02160 merged commit 95015cf into refactor/#82-multi_module Aug 12, 2023
Copy link
Member

@ddongseop ddongseop left a comment

Choose a reason for hiding this comment

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

수고했으,,,,!!

@jun02160 jun02160 linked an issue Aug 12, 2023 that may be closed by this pull request
5 tasks
@ddongseop ddongseop deleted the feat/#87-sqs_push_notification branch August 12, 2023 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feat 새로운 기능 추가 예준🍒 🔥 Pull Request PR 날림
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] SQS를 적용한 푸시알림 기능 개선
2 participants