[Notification] 활성 대화창의 알림 전송 버그 수정

[loi2016]

PR 요약

대화 활성 여부가 항상 false 처리되고 대화 중에도 SSE 알림이 전송되어 이를 수정

체크리스트

• 이 PR과 관련된 이슈가 있나요? ([Notification] 활성 대화창의 알림 전송 버그 수정 #125 )
• 기능/버그 수정 테스트 완료
• 코드 리뷰 완료

상세 설명

• WebSocket principal을 DirectMessageEventListener.extraUserId()에서 MoplUserDetails로만 처리하여 userId가 null로만 들어옴
• isSubscribed()가 항상 false가 되어 이벤트 생성 및 알림 전송이 되어 미확인 알림 카운트 증가/DB 알림 삭제가 이뤄지고, 알림 배지 클릭 시 정합성이 맞춰지면서 다시 카운트가 감소하는 현상이 존재
• UUID, String 등도 처리할 수 있도록 메서드에 경우의 수 추가

변경된 파일

ReviewService

• 리뷰 알림이 과하게 길어지는 것을 대비해 20자까지만 출력하도록 변경

DirectMessageSubscriptionRegistry

• 코드 상 수정 없음

DirectMessageEventListener

• 컨트롤러 및 인증 사양에 맞게 principal 처리 경우의 수 추가

추가된 파일

DirectMessageReadService

• 리스너에서 호출하여 알림 삭제를 진행하는 서비스

제거된 파일

• NULL

검증 단계

추가 코멘트

• (추후 수정이 필요한 문제, 원인 추정) DM 컨트롤러에서 초기 방식으로 Authentication을 처리하고 이것이 Jwt 인터셉터와 연결되어 원인을 해결하려면 많은 작업 필요할 것으로 예상