Skip to content

Conversation

@uhm0311
Copy link
Collaborator

@uhm0311 uhm0311 commented Nov 13, 2025

🔗 Related Issue

reconnectBlocked 관련하여 아래 버그가 있다.

  • 처음 재연결 시 (immediate reconnect 가정)
    • setupForAuth()에서 reconnectBlocked 생성
    • authComplete()에서 reconnectBlocked에 있는 op들을 처리함. (reconnectBlocked는 그대로 유지)
  • 다음 재연결 시 (delayed reconnect 가정)
    • delayed reconnect에서는 writeQ와 inputQ에 있는 op들을 cancel 시킴
    • setupForAuth()에서 reconnectBlocked을 새로 생성하지 않음 (writeQ와 inputQ가 empty이므로)
    • authComplete()에서 이전에 생성된 reconnectBlocked에 있던 op들을 다시 처리함 (버그 부분)

⌨️ What I did

  • authComplete 완료 후 reconnectBlocked 값을 null로 초기화합니다.

@uhm0311 uhm0311 requested a review from jhpark816 November 13, 2025 06:42
@jhpark816
Copy link
Collaborator

@uhm0311
문제가 되는 경우에 대한 설명을 PR 코멘트에 추가 바랍니다.

@uhm0311
Copy link
Collaborator Author

uhm0311 commented Nov 13, 2025

추가했습니다.

@jhpark816
Copy link
Collaborator

@uhm0311 코멘트를 다시 정리했습니다.

} else {
inputQueue.addAll(reconnectBlocked);
}
reconnectBlocked = null;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reconnectBlocked가 empty()인 경우는 null로 clear하지 않는데요.
문제가 없으므로 현재 PR을 그대로 merge합니다.

@jhpark816
Copy link
Collaborator

@oliviarla @brido4125
수정 사항을 확인 바랍니다.

@jhpark816 jhpark816 merged commit fd5c2fd into naver:develop Nov 13, 2025
2 checks passed
@uhm0311 uhm0311 deleted the uhm0311/f/2 branch November 13, 2025 07:13
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