[feat] 재연결 로직 구현 #102

sehee123 · 2025-07-21T12:25:32Z

🛰️ Issue Number

[feat] 방 재연결 관리 #34

🪐 작업 내용

새로고침, 브라우저 닫기 등 비정상 disconnect에 대한 보완
기존에 Room에서 관리 되던 Map<UserId, SessionId> map 을 새로 만든 SessionService에서 관리하도록 변경

전체적인 흐름

disconnect 이벤트리스너에서 연결끊김을 감지한 후 room의 Player정보를 disconnect로 변경
5초의 유예시간 후 connected가 되었으면 ~님이 재연결 되었습니다. systemNotice 전송 (추후 개인에게만 전송으로 변경예정)
5초 후에도 disconnected 라면 게임 진행중이 아니면 exit 처리(이미 exit 처리가 된 정상로직이 아닐경우를 판단하여 처리)
connect 이벤트 리스너에서는 새로 연결된 sessionId를 put 함
subscribe 이벤트 리스너에서 roomId를 알 수있으므로 roomId 와 매핑을 위한 map 에 sessionId를 put
재연견을 판별하여 player정보에서 sessionId를 갱신해줌

oldSessionId 값을 관리하기 위해 userIdLatestSession 를 생성

클라이언트는 새로고침시 바로 disconnected 및 subscribe 할 것으로 가정하고 구현

📚 Reference

✅ Check List

코드가 정상적으로 컴파일되나요?
테스트 코드를 통과했나요?
merge할 브랜치의 위치를 확인했나요?
Label을 지정했나요?

# Conflicts: # backend/src/main/java/io/f1/backend/domain/game/app/RoomService.java

# Conflicts: # backend/src/main/java/io/f1/backend/domain/game/app/RoomService.java # backend/src/main/java/io/f1/backend/domain/game/websocket/GameSocketController.java # backend/src/main/java/io/f1/backend/global/exception/errorcode/RoomErrorCode.java # backend/src/test/java/io/f1/backend/domain/game/app/RoomServiceTests.java

# Conflicts: # backend/src/main/java/io/f1/backend/global/config/StompChannelInterceptor.java

# Conflicts: # backend/src/main/java/io/f1/backend/domain/game/dto/RoomEventType.java # backend/src/main/java/io/f1/backend/domain/game/mapper/RoomMapper.java # backend/src/main/java/io/f1/backend/domain/game/model/Room.java # backend/src/main/java/io/f1/backend/domain/game/websocket/controller/GameSocketController.java

jiwon1217 · 2025-07-21T15:22:40Z

backend/src/main/java/io/f1/backend/domain/auth/dto/CurrentUserAndAdminResponse.java

                userPrincipal.getUserNickname(),
-                UserPrincipal.ROLE_USER);
+                UserPrincipal.ROLE_USER,
+                userPrincipal.getName());


API 명세서에 /auth/me response를 수정했습니다. 해당 PR이 merge가 되면 재연결 로직이 구현됨에 따라 명세서가 수정되었다고 말씀드리면 될 것 같습니다 !

이부분은 재연결때문에 추가한건 아니였고, 추후에 1:1 응답에서 추가될 예정이라서 그부분 구현하고나서 제가 말씀드리겠습니다!
감사합니다.. 💪

jiwon1217 · 2025-07-21T15:27:38Z

backend/src/main/java/io/f1/backend/domain/game/mapper/RoomMapper.java

        } else if (roomEventType == RoomEventType.EXIT) {
            message = " 님이 퇴장하셨습니다";
+        } else if (roomEventType == RoomEventType.RECONNECT) {
+            message = " 님이 재연결 되었습니다.";


이 부분은 은주님 PR에서 SystemNoticeMessage enum으로 분리한 것으로 알고 있습니다. 확인부탁드립니다 ~!

넵 충돌 해결하면서 수정했습니다! 정리 후에 push하겠습니닷!

dlsrks1021 · 2025-07-21T14:31:26Z

backend/src/main/java/io/f1/backend/domain/game/model/Room.java

+        playerSessionMap.get(sessionId).updateState(newState);
+    }
+
+    public boolean isReconnectTarget(String sessionId) {


[L4-변경제안]
isExit 메서드와 isReconnectTarget은 보수 관계인 것 같습니다.
isExit 메서드로 통합한 이후 isReconnectTarget 사용부를 !isExit()으로 대체 가능할 것 같습니다.

감사합니다.. 하나로 통일해서 올리겠습니다.

backend/src/main/java/io/f1/backend/domain/game/websocket/Service/SessionService.java

backend/src/main/java/io/f1/backend/domain/game/app/RoomService.java

silver-eunjoo · 2025-07-22T02:14:35Z

.../src/main/java/io/f1/backend/domain/game/websocket/eventlistener/WebsocketEventListener.java

+
+        Message<?> message = event.getMessage();
+
+        StompHeaderAccessor headerAccessor = StompHeaderAccessor.wrap(message);


여기서는 WebSocketUtils를 사용하지 않은 이유가 있을까요?

room 구독만을 생각하지않고 구독주소의 확장성때문에 현재 있는 고정값을 사용하지 않았습니닷 !

silver-eunjoo · 2025-07-22T02:20:40Z

.../src/main/java/io/f1/backend/domain/game/websocket/eventlistener/WebsocketEventListener.java

+        String sessionId = getSessionId(message);
+        UserPrincipal principal = getSessionUser(message);
+
+        String destination = headerAccessor.getDestination();


만약, 이 로직을 위해 따로 util 을 사용하지 않았다면 room 을 알기 위해 사용하는 메서드 네임을 getRoomSubscriptionDestination 이런 식으로 해서 util로 빼는 건 어떨지 제안드려봅니다..!

util로 빼야지~ 생각만해버렸네요 .. ㅎㅎ 감사합니다! 💪

silver-eunjoo · 2025-07-22T04:19:10Z

backend/src/main/java/io/f1/backend/domain/game/websocket/service/SessionService.java

+    private final Map<String, Long> sessionIdRoom = new ConcurrentHashMap<>();
+    private final Map<Long, String> userIdSession = new ConcurrentHashMap<>();
+    private final Map<Long, String> userIdLatestSession = new ConcurrentHashMap<>();
+    private final ScheduledExecutorService scheduler = Executors.newScheduledThreadPool(1);


쓰레드풀의 개수를 1로 두면 재연결을 처리할 수 있는 하나의 스케줄러가 재연결을 하나씩 순서대로 처리하게 되어 비효율적일 것 같다는 생각이 듭니다 ! 쓰레드 풀의 개수를 늘려서 재연결을 여러 개씩 병렬적으로 처리할 수 있도록 하거나, 재연결을 처리해야하는 주체인 플레이어에게 하나씩 배정해줄 수 있을 것 같습니다 !

넵! 위에 강현님 의견이랑 같이해서 고민해보겠습니다 감사합니다!

silver-eunjoo · 2025-07-22T04:21:43Z

backend/src/main/java/io/f1/backend/domain/game/websocket/service/SessionService.java

+
+        Long userId = principal.getUserId();
+
+        if (userIdLatestSession.get(userId) != null) {


[L5-참고의견]
이중 if문을 없애기 위해 null일 때는 return을 해주면서 흐름을 한 번 끊고, null이 아닐 때의 로직을 처리하는 방식도 좋을 것 같습니다.

silver-eunjoo · 2025-07-22T04:31:48Z

backend/src/main/java/io/f1/backend/global/config/WebSocketConfig.java

    @Override
    public void configureMessageBroker(MessageBrokerRegistry registry) {
-        registry.enableSimpleBroker("/sub");
+        registry.enableSimpleBroker("/sub", "/queue");


제가 설명을 놓친 것 같은데.. 혹시 이 부분이 어떤 내용인지 알 수 있을까요?

1:1 메세지를 위한 구독채널이 추가가 되어야해서 수정된 부분입니다! 사실 이번 pr에는 포함안되어도 되는 부분이긴합니닷.

# Conflicts: # backend/src/main/java/io/f1/backend/domain/game/websocket/service/SessionService.java

silver-eunjoo · 2025-07-22T08:53:44Z

backend/src/main/java/io/f1/backend/domain/game/app/RoomService.java

    private void removePlayer(Room room, String sessionId, Player removePlayer) {
        room.removeUserId(removePlayer.getId());
        room.removeSessionId(sessionId);
+        room.removeValidatedUserIds(removePlayer.getId());


[L5-참고의견]
얘도 아이디 하나를 지워주는 것이니 복수형보다는 removeValidatedUserId 단수형으로 수정되면 좋을 것 같습니다 !

# Conflicts: # backend/src/main/java/io/f1/backend/domain/game/websocket/service/SessionService.java

jiwon1217

고생하셨습니다

# Conflicts: # backend/src/main/resources/application.yml

silver-eunjoo

재연결 로직 .. 대단해요..! 고생하셨습니다 !! 🔥

LimKangHyun

고생하셨습니다!

sehee123 and others added 16 commits July 17, 2025 19:24

♻️ destination 생성 위치 변경

d32fa2c

chore: Java 스타일 수정

4a87b5b

✨ subscribe 이벤트 리스너 구현

2ebc45b

Merge remote-tracking branch 'origin/dev' into feat/34

22e2dce

# Conflicts: # backend/src/main/java/io/f1/backend/domain/game/app/RoomService.java

♻️ 재연결 중간 커밋

fb25877

Merge remote-tracking branch 'origin/dev' into feat/34

7be49b7

# Conflicts: # backend/src/main/java/io/f1/backend/global/config/StompChannelInterceptor.java

✨ 재연결 로직 완료

51efbed

chore: Java 스타일 수정

5ba8a13

♻️ 테스트값 원복

07da462

♻️ 재입장 시 기본 방 세팅 값 브로드캐스트

6ce1897

chore: Java 스타일 수정

527a6c5

♻️ 중복검사 제거

bde0694

Merge remote-tracking branch 'origin/feat/34' into feat/34

bdc1700

♻️ remove 위치 변경

4abe282

chore: Java 스타일 수정

849ef71

sehee123 self-assigned this Jul 21, 2025

sehee123 added the enhancement New feature or request label Jul 21, 2025

sehee123 linked an issue Jul 21, 2025 that may be closed by this pull request

[feat] 방 재연결 관리 #34

Closed

1 task

sehee123 added 3 commits July 21, 2025 21:31

♻️ test 수정

bc3fd2b

Merge remote-tracking branch 'origin/feat/34' into feat/34

435d8f0

jiwon1217 reviewed Jul 21, 2025

View reviewed changes

sehee123 and others added 3 commits July 22, 2025 00:52

♻️ NPE 보완

781c283

♻️ RoomEventType reconnect 추가

fd9baf5

chore: Java 스타일 수정

fd9aff1

dlsrks1021 reviewed Jul 21, 2025

View reviewed changes

sehee123 added 2 commits July 22, 2025 01:24

♻️ 리뷰내용 반영

7bd4453

Merge remote-tracking branch 'origin/feat/34' into feat/34

e3dcb00

silver-eunjoo reviewed Jul 22, 2025

View reviewed changes

sehee123 and others added 3 commits July 22, 2025 12:52

♻️ 리뷰 반영 - util 추가, 메소드명 수정, 널세이프수정

3914981

Merge remote-tracking branch 'origin/feat/34' into feat/34

e8616f4

chore: Java 스타일 수정

f1dd3a2

silver-eunjoo reviewed Jul 22, 2025

View reviewed changes

sehee123 and others added 4 commits July 22, 2025 17:23

🐛 oldSession 관리 순서 변경

ba3e466

♻️ 테스트 수정

a568032

Merge remote-tracking branch 'origin/feat/34' into feat/34

e25f2f2

# Conflicts: # backend/src/main/java/io/f1/backend/domain/game/websocket/service/SessionService.java

chore: Java 스타일 수정

cce37c4

silver-eunjoo reviewed Jul 22, 2025

View reviewed changes

sehee123 and others added 3 commits July 22, 2025 18:05

♻️ 리뷰 반영 s 삭제

3aef18f

Merge remote-tracking branch 'origin/feat/34' into feat/34

6a5ad37

# Conflicts: # backend/src/main/java/io/f1/backend/domain/game/websocket/service/SessionService.java

chore: Java 스타일 수정

c506712

jiwon1217 approved these changes Jul 22, 2025

View reviewed changes

sehee123 and others added 6 commits July 22, 2025 22:28

♻️ 재입장시 connect 상태 변경, 응답 추가

5a1b865

♻️ 스레드 풀 1-> 2 변경

6424848

Merge remote-tracking branch 'origin/feat/34' into feat/34

0e644b7

chore: Java 스타일 수정

b408205

Merge remote-tracking branch 'origin/dev' into feat/34

cbb9a72

# Conflicts: # backend/src/main/resources/application.yml

Merge remote-tracking branch 'origin/feat/34' into feat/34

fc37e91

silver-eunjoo approved these changes Jul 23, 2025

View reviewed changes

sehee123 and others added 2 commits July 23, 2025 13:42

✅ test 코드추가

f9606ba

chore: Java 스타일 수정

83544c5

LimKangHyun approved these changes Jul 23, 2025

View reviewed changes

dlsrks1021 approved these changes Jul 23, 2025

View reviewed changes

sehee123 merged commit 1f88146 into dev Jul 23, 2025

sehee123 deleted the feat/34 branch July 23, 2025 05:16


		Message<?> message = event.getMessage();

		StompHeaderAccessor headerAccessor = StompHeaderAccessor.wrap(message);


		Long userId = principal.getUserId();

		if (userIdLatestSession.get(userId) != null) {

[feat] 재연결 로직 구현 #102

[feat] 재연결 로직 구현 #102

Uh oh!

Conversation

sehee123 commented Jul 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🛰️ Issue Number

🪐 작업 내용

전체적인 흐름

📚 Reference

✅ Check List

Uh oh!

jiwon1217 Jul 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sehee123 Jul 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jiwon1217 left a comment

Choose a reason for hiding this comment

Uh oh!

silver-eunjoo left a comment

Choose a reason for hiding this comment

Uh oh!

LimKangHyun left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

sehee123 commented Jul 21, 2025 •

edited

Loading

jiwon1217 Jul 21, 2025 •

edited

Loading

sehee123 Jul 22, 2025 •

edited

Loading