[Chat] 채팅방에 Event가 일어났을 때 해당 채팅방을 구독하는 사용자들에게 보내기 #284
Conversation
minisundev
changed the title
| @@ -22,6 +22,7 @@ class ChatController( | |||
| private val authClient: AuthClient, | |||
| private val serverClient: ServerClient, | |||
| ) { | |||
| @Deprecated("WebSocketChatController를 대신 사용") | |||
There was a problem hiding this comment.
해당 annotation은 어떤 역할을 하는 건가요?!
There was a problem hiding this comment.
@deprecated 어노테이션은 이 메서드가 더 이상 사용되지 않음을 명확히 하기 위한 표시입니다! WebSocketChatController가 동일한 기능을 대체하고 있지만, 이 메서드가 구현된 로직을 보존해둔 이유는 기존 구현을 참고해야 할 경우도 있고, 특정 케이스가 예상대로 동작하지 않을 때 비교 자료로 쓰기 위해서 입니다! 주석을 통해 대체된 위치를 명시했기 때문에 혼동 없이 코드 유지와 개선에 도움이 될 것이라 판단했습니다!
| else -> message | ||
| } | ||
| } | ||
| } | ||
| } | ||
|
|
||
| private fun handleConnectMessage(message: Message<*>): Message<*> { | ||
| private fun authenticateAndSetPrincipal(message: Message<*>): Message<*> { |
There was a problem hiding this comment.
해당 메서드에서 변경된 내용이 없는 것 같은데 메서드명을 변경하신 이유가 있는지 궁금합니다!
There was a problem hiding this comment.
원래 STOMP 메시지 타입이 CONNECT일 때는 유저 정보를 세션에 저장하고
SUBSCRIBE일 때는 유저가 해당 채팅방/서버에 권한이 있는지 확인하는 과정을 거쳤기 때문에 메서드명을 handleConnectMessage, handleSubscribeMessage로 했었습니다!
하지만 STOMP 메시지 타입이 MESSAGE일 때에도 권한이 있는지 확인해서 handleSubscribeMessage를 사용하게 되어 handleSubscribe의 메서드 명을 범용적으로 변경하는 과정에서 STOMP 메시지 타입으로 분류했던 메서드(handleConenctMessage, handleSubscribeMessage) 명을 모두 수정하게 되었습니다!
#️⃣연관된 이슈
close #278
📝작업 내용