Skip to content

채팅 시작 시 빈 채팅방 누락 및 WebSocket 인증 보완#248

Merged
Goder-0 merged 1 commit into
mainfrom
fix/#247-chat-start-error
Jun 24, 2026
Merged

채팅 시작 시 빈 채팅방 누락 및 WebSocket 인증 보완#248
Goder-0 merged 1 commit into
mainfrom
fix/#247-chat-start-error

Conversation

@Goder-0

@Goder-0 Goder-0 commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Overview

  • 채팅 시작 시 응답이 없고 채팅방이 생성되지 않은 것처럼 보이던 문제를 수정했습니다.
  • 실제 원인은 새 채팅방이 메시지 없이 생성된 뒤 목록에서 누락되는 문제와, SockJS 환경의 WebSocket 인증 불안정이 겹친 것이었습니다.

Close: #247

Changes

  • 메시지가 없는 새 채팅방도 채팅 목록에 포함되도록 조회 쿼리를 수정했습니다.
  • 마지막 메시지가 없는 경우 chat.createdAt 기준으로 정렬되도록 보완했습니다.
  • SockJS handshake 단계에서 쿠키의 accessToken 을 STOMP 인증에 전달하도록 처리했습니다.
  • 채팅 조회 실패 시 무응답처럼 끝나지 않고 에러 payload 를 내려주도록 보완했습니다.
  • 빈 채팅방 노출, 쿠키 기반 WebSocket 인증, 초기 조회 실패 처리에 대한 테스트를 추가했습니다.

@Goder-0 Goder-0 changed the title [fix] 채팅 시작 실패 흐름 수정 (#247) 채팅 시작 시 빈 채팅방 누락 및 WebSocket 인증 보완 Jun 21, 2026
@Goder-0 Goder-0 force-pushed the fix/#247-chat-start-error branch from 1f8e04d to 74c6373 Compare June 21, 2026 13:06
@Goder-0 Goder-0 marked this pull request as ready for review June 21, 2026 13:07
@Goder-0 Goder-0 requested review from ckdals4600 and minibr June 21, 2026 13:07
@Goder-0 Goder-0 force-pushed the fix/#247-chat-start-error branch 2 times, most recently from 9e594d4 to a5e9aad Compare June 24, 2026 12:58
@github-actions

github-actions Bot commented Jun 24, 2026

Copy link
Copy Markdown

📊 코드 커버리지 리포트

Overall Project 94.07% -0.29% 🍏
Files changed 83.87% 🍏

File Coverage
ChatFacade.java 92.34% -7.18% 🍏

SockJS handshake 경로를 인증 예외로 열어 채팅방 생성 후 첫 질문이 소켓으로 전송되도록 합니다.

메시지가 아직 없는 채팅방도 목록에서 조회되도록 유지해 실패 상태를 확인할 수 있게 합니다.
@Goder-0 Goder-0 force-pushed the fix/#247-chat-start-error branch from a5e9aad to 764a07e Compare June 24, 2026 14:16
@Goder-0 Goder-0 merged commit 6e85854 into main Jun 24, 2026
1 check passed
@Goder-0 Goder-0 deleted the fix/#247-chat-start-error branch June 24, 2026 14:20
@ckdals4600 ckdals4600 restored the fix/#247-chat-start-error branch June 25, 2026 13:54
@ckdals4600 ckdals4600 deleted the fix/#247-chat-start-error branch June 25, 2026 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

채팅 시작 시 응답이 없고 새 채팅방이 목록에 보이지 않는 문제

2 participants