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

♻️ 구독자가 발행된 이벤트를 신뢰하도록 불필요한 검증 로직 제거 #835

Open
uwoobeat opened this issue Jan 6, 2025 · 0 comments
Labels
♻️ refactor 기능 개선

Comments

@uwoobeat
Copy link
Member

uwoobeat commented Jan 6, 2025

📌 Description

  • 몇몇 이벤트 핸들러 혹은 핸들러가 호출하는 서비스에서 '이미 발생한' 이벤트의 진실 여부를 불필요하게 검증하는 경우 있음
  • 가령 주문 완료 이벤트가 있다면 -> 구독자에서 실제로 주문이 완료되었는지 더블 체크하는 식
  • 설사 상태가 잘못되었다 한들 그걸 검증하고 보장하는 것은 발행자의 책임
  • 구독자가 해당 상태를 검증하게 된다는 것은 발행자와 도메인 로직을 알아야 한다는 것이므로 결합도 증가 -> 만약 검증에 대한 정책이 바뀌면 모든 구독자 로직을 수정해야 하는 문제
  • 또한 모든 구독자가 발행자를 검증하면 불필요한 추가 검증 발생 (구독자마자 검증 중복 발생)
  • 따라서 구독자는 발행자의 상태를 신뢰하고 (즉 해당 이벤트가 진실일 것이라고 100% 믿고) 로직을 수행해야 함
@uwoobeat uwoobeat added the ♻️ refactor 기능 개선 label Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor 기능 개선
Projects
None yet
Development

No branches or pull requests

1 participant