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] rabbitmq deadletter 처리 #89

Merged
merged 2 commits into from
Mar 11, 2024
Merged

Conversation

ohksj77
Copy link
Member

@ohksj77 ohksj77 commented Mar 11, 2024

추가/수정한 기능 설명

  • 다음과 같은 deadletter 전략 적용
    1. 기존 서비스의 큐에 데드레터 발생시 모두 동일한 데드레터 큐로 전달
    2. 로깅 및 슬랙 알림
  • 이유
    • 여러 단계의 처리는 없으므로 단계를 저장하는 것은 불필요
    • 이미 슬랙 사용중이므로 다른 알림 시스템 보다는 슬랙 사용
  • 추후 개선사항
    • 데드레터 큐에서 에러 발생시 대처 방안 수립
    • 데드레터 정보가 방대해질 경우 DB에 저장 (현재는 단순한 정보이므로 모두 슬랙에 전달)

특이사항

  • 없음

check list

  • 모든 단위 테스트를 돌려보고 기존에 작동하던 테스트에 영향이 없는 것을 확인했나요?
  • 추가/수정사항을 설명했나요?

@ohksj77 ohksj77 added the feat 기능 추가 label Mar 11, 2024
@ohksj77 ohksj77 requested a review from jinjoo-lab March 11, 2024 07:45
@ohksj77 ohksj77 self-assigned this Mar 11, 2024
@ohksj77
Copy link
Member Author

ohksj77 commented Mar 11, 2024

Test Coverage

File Coverage [40%]
FcmConsumer.java 58.82% 🍏
DeadLetterConsumer.java 32.56%
Total Project Coverage 57.41% 🍏

@ohksj77 ohksj77 merged commit d05d4fe into master Mar 11, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant