14KGun commented 1 year ago

Describe the bug

여러 개의 채팅방에 동시에 참여하고 있는 경우 다른 방 채팅이 옵니다. 채팅방을 나갔다 들어오면 그 채팅은 사라지긴 합니다.

Screenshots

withSang commented 1 year ago

서버에서는 roomId 구분 없이 Socket.io에 연결된 방 참여중인 모든 사용자에게 새 채팅 메시지 이벤트를 발생시킵니다(해당 코드)

두 가지 해결방법이 있을 것 같습니다!

클라이언트에서 새 채팅 메시지 이벤트가 발생했을 때 새 채팅 메시지의 roomId가 현재 참여중인 방의 roomId와 같을 때에만 메시지를 표시하는 검증 단계를 추가합니다. 해당 채팅방을 지금 켜놓지 않은 사용자에게도 메시지가 전송되기 때문에 클라이언트 측면에서 불필요한 처리가 발생합니다. 아래 함수에 추가할 수 있습니다. https://github.com/sparcs-kaist/taxi-front/blob/8066b84941cdd90a5e21f614929b063aa77727dd/src/pages/Chatting/index.jsx#L175-L197
서버의 emitChatEvent 함수를 수정하여 방에 참여중이고 현재 해당 채팅방을 켜놓은 사용자들에게만 채팅 메시지를 전송합니다. 서버에서 Redis로 캐시를 관리하면서 각 사용자가 현재 켜놓은 채팅방의 roomId를 저장할 수 있습니다.

withSang commented 1 year ago

@14KGun 헐 이미 커밋을 날리셨군요 빠르세요 ㄷㄷ

14KGun commented 1 year ago

@withSang @chlehdwon

근데 클라이언트가 메시지의 roomId를 알 방법이 없는 것 같은데, 해당 함수에다가 roomId를 추가햐야 할 것 같아요.

둘이 필드를 통일시켜야 할 듯 합니당 ㅜ

(back도 typescript 도입을... ㅜ)