Hotfix/#127: 안읽은 메시지 버그 수정

[hyunn522]

📝 요약

• resolved Hotfix: 안읽은 메시지 버그 수정 #127

🔖 변경 사항

socket disconnect 시에 세션에 저장된 userId와 chatRoomId를 통해, 유저가 채팅방을 떠난 시간을 저장하는 방식에서
유저가 채팅방을 떠날 때마다 http 요청을 보내면 서버에서 시간을 저장하는 로직으로 변경했습니다
비정상적인 disconnect가 발생한 경우에는 세션에 userId와 chatRoomId가 저장되어있지 않는 경우도 있는 것 같아서
해당 경우엔 userId로 user를 찾지 못하므로 불필요한 서버 오류가 많아지는 것 같아서 좀 더 확실한 방식을 선택했습니다
좋은 의견 있으시면 남겨주세요

✅ 리뷰 요구사항

📸 확인 방법 (선택)

---

📌 PR 진행 시 이러한 점들을 참고해 주세요

* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

[seongjunnoh]

좋은거 같습니다!
저도 저번 팀플에서 이벤트 리스너를 이용해서 disconnect를 식별하는 방식을 이용했었는데, 아무래도 disconnect 가능한 경우도 많고, 해당 이벤트가 발생할때 저장해야할 데이터들이 날라가는 이슈가 발생하더라고요
저는 마지막 메시지를 채팅방 외부에서 볼 수 있도록 하는 기능은 없어서 SimpMessageHeaderAccessor 에 필요한 데이터들을 저장해놓고, disconnect 이벤트 발생시 동작하는 코드에서 여기에 저장한 데이터들을 다시 꺼내서 사용하는 방식을 사용했었긴 합니다!
그런데 저는 정적인 데이터들을 저장할 필요가 있었던 것이었고, 현재는 아무래도 동적으로 변하는 채팅을 저장해야하니까 이 방식보다는 서현님이 작업하신 방법이 더 나은거 같네요

[hyunn522]

좋은거 같습니다! 저도 저번 팀플에서 이벤트 리스너를 이용해서 disconnect를 식별하는 방식을 이용했었는데, 아무래도 disconnect 가능한 경우도 많고, 해당 이벤트가 발생할때 저장해야할 데이터들이 날라가는 이슈가 발생하더라고요 저는 마지막 메시지를 채팅방 외부에서 볼 수 있도록 하는 기능은 없어서 SimpMessageHeaderAccessor 에 필요한 데이터들을 저장해놓고, disconnect 이벤트 발생시 동작하는 코드에서 여기에 저장한 데이터들을 다시 꺼내서 사용하는 방식을 사용했었긴 합니다! 그런데 저는 정적인 데이터들을 저장할 필요가 있었던 것이었고, 현재는 아무래도 동적으로 변하는 채팅을 저장해야하니까 이 방식보다는 서현님이 작업하신 방법이 더 나은거 같네요

네 왜인지 모르겠는데 데이터가 날라가는 이슈 때문에 저도 이 방식을 택했습니다! 그럼 그대로 진행하겠습니다아