Skip to content

Block on TLS auth in NetworkHandler#1748

Merged
kevin-montrose merged 2 commits into
mainfrom
users/kmontrose/networkHandlerStartBlocking
Apr 29, 2026
Merged

Block on TLS auth in NetworkHandler#1748
kevin-montrose merged 2 commits into
mainfrom
users/kmontrose/networkHandlerStartBlocking

Conversation

@kevin-montrose
Copy link
Copy Markdown
Contributor

@kevin-montrose kevin-montrose commented Apr 29, 2026

Follow up on #1714 - block in NetworkHandler.Start() methods on TLS auth.

TODO:

Copilot AI review requested due to automatic review settings April 29, 2026 16:47
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR follows up on #1714 by making TLS authentication completion a synchronous guarantee in NetworkHandler.Start(...), so callers no longer need to spin-wait for SslStream authentication before using the connection.

Changes:

  • Updated NetworkHandler.Start(...) (client/server TLS overloads) to synchronously wait for TLS authentication via AsyncUtils.BlockingWait(...).
  • Removed IsAuthenticated(out Exception fault) / authFault and deleted client-side spin-wait loops that previously polled for TLS auth completion.
  • Updated benchmark embedded server path to stop spinning on TLS auth.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
libs/common/Networking/NetworkHandler.cs Makes sync Start(...) wait for TLS auth; removes IsAuthenticated/authFault; updates related XML docs.
libs/common/LightClient.cs Removes TLS-auth spin loop after networkHandler.Start(...).
libs/client/GarnetClient.cs Removes TLS-auth spin loop after networkHandler.Start(...).
libs/client/ClientSession/GarnetClientSession.cs Removes TLS-auth spin loop after networkHandler.Start(...).
benchmark/BDN.benchmark/Embedded/GarnetServerEmbedded.cs Removes TLS-auth spin loop after handler.Start(...).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread libs/common/Networking/NetworkHandler.cs
Comment thread libs/common/Networking/NetworkHandler.cs Outdated
Comment thread libs/common/Networking/NetworkHandler.cs Outdated
@kevin-montrose kevin-montrose merged commit e17c12a into main Apr 29, 2026
47 of 49 checks passed
@kevin-montrose kevin-montrose deleted the users/kmontrose/networkHandlerStartBlocking branch April 29, 2026 19:23
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