Skip to content

INTERNAL: Add logging for NOT_SUPPORTED response while authenticating#1036

Merged
jhpark816 merged 1 commit intonaver:developfrom
uhm0311:uhm0311/f/1
Jan 19, 2026
Merged

INTERNAL: Add logging for NOT_SUPPORTED response while authenticating#1036
jhpark816 merged 1 commit intonaver:developfrom
uhm0311:uhm0311/f/1

Conversation

@uhm0311
Copy link
Collaborator

@uhm0311 uhm0311 commented Jan 14, 2026

🔗 Related Issue

⌨️ What I did

  • 서버가 인증 기능을 제공하지 않아 인증 과정을 생략한다는 내용의 로깅을 추가합니다.

@uhm0311 uhm0311 requested a review from jhpark816 January 14, 2026 09:28
Copy link
Collaborator

@jhpark816 jhpark816 left a comment

Choose a reason for hiding this comment

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

리뷰 완료

} else if ("NOT_SUPPORTED".equals(msg)) {
authDone = true;
node.authComplete(true);
getLogger().warn("Authentication not supported by server, skipping auth flow");
Copy link
Collaborator

Choose a reason for hiding this comment

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

@namsic
코드로 보니, node 정보도 추가하는 것이 좋을 것 같습니다.
확인해 주세요.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Authenticated to " + node.getSocketAddress() 로그와 유사하게 아래와 같이 할 수 있어 보입니다.

"Authentication not supported by server, skipping auth flow: " + node.getSocketAddress()

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

수정했습니다.

@jhpark816 jhpark816 merged commit 93196d6 into naver:develop Jan 19, 2026
2 checks passed
@uhm0311 uhm0311 deleted the uhm0311/f/1 branch January 26, 2026 02:21
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.

3 participants