[Refactor] 공용 메시지 브로커 구현 및 적용 #16
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Pub/Sub 패턴 적용을 위한 메시지 큐 구현
- MessageQueue 구현체 Nest 전역 모듈 등록
- 구현 변경으로 인한 메시지큐 관련 클래스 제거
- 간단한 pub-sub 구현
- 간단한 pub-sub 클래스 chat에 적용
- publish 반환 타입 수정
- 메시지 브로커 클래스에 토픽에 대한 제네릭 타입 추가 - subscribe 메소드 반환값을 unsubscribe 로 변경 - 퀴즈를 떠났을 때 채팅에서도 unsubscribe 되도록 수정
- 브로커 의존성 주입 과정에서 publisher 주입 안되는 문제 수정
- RxJS를 활용한 메시지 브로커 구현
- ReactiveMessageBroker 적용 및 테스트 완료
- 채팅 기능 메시지 보내기 핸들러 형식 수정
- play 웹소켓 응답 처리 pub/sub 적용 - 불필요한 로직 삭제 - 일부 코드 개선
- pub-sub 이름을 broker로 변경 - play, chat 모듈에 변경사항 반영 - 디렉터리 위치 변경
- 유연한 메시지 처리를 위한 Message를 제네릭으로 변경 - Message 제네릭 변경 반영
- 브로커 제네릭 변경으로 인한 메시지 브로드캐스팅 로직 변경 - SendEvent 인터페이스 sender 속성 추가 - SendEvent 인터페이스 변경으로 인한 핸들러 처리 로직 변경
- chat 서비스 브로커 타입 선언 변경
- broker 모듈 공용 모듈로 위치 변경 - message-broker undefined narrowing 추가 - reactive-message-broker.ts error 타입 추가
- 공용 브로커 모듈 내보내기 추가
- broker를 위한 rxjs 의존성 추가
- broker 관련 참조를 shared로 변경
- 생성자 인자를 readonly로 변경
- 모노레포 shared 패키지 rxjs 의존성 추가
- broker 도입을 통한 사용자 정보 의존성 분리
- 누락된 chat 핸들러 복구
codemario318
changed the title
- subscribe의 불필요한 subscriberId 인자 제거 - MessageBroker를 위한 Subscriber 인터페이스 선언 - MessageBroker subscriber 저장을 배열로 변경 - subscriber의 Id를 생성하도록 변경(uuid) - ReactiveMessageBroker의 subscribe 변경 반영
- playGateway의 broker 변경 사항 반영 - chatService의 broker 변경 사항 반영
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔍️ 이 PR을 통해 해결하려는 문제가 무엇인가요?
✨ 이 PR에서 핵심적으로 변경된 사항은 무엇일까요?
🙏 Reviewer 분들이 이런 부분을 신경써서 봐 주시면 좋겠어요
자세한 내용은 Wiki - 개발문서 참고해주세요 😄