Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SNI extension when connecting with TLS #786

Closed
thierryba opened this issue Aug 9, 2024 · 0 comments · Fixed by #787
Closed

Add SNI extension when connecting with TLS #786

thierryba opened this issue Aug 9, 2024 · 0 comments · Fixed by #787
Labels
proposal Enhancement idea or proposal

Comments

@thierryba
Copy link
Contributor

Proposed change

It would be nice (and consistent with nats-py and the nats-server to set the SNI extension when connecting via TLS. That allows for load balancing to work properly in the cloud.

Use case

We hae nats instances for each of our customers. We are on the cloud. Today we have to use 1 load balancer per client. With the Fix to issue 780 and the addition of handshake_first, the last issue is that the Client Hello is missing the SNI extension. That can be done with 1 simple call to OpenSSL

Contribution

definitely.

@thierryba thierryba added the proposal Enhancement idea or proposal label Aug 9, 2024
@levb levb closed this as completed in #787 Aug 12, 2024
umesh3034 pushed a commit to umesh3034/cnats that referenced this issue Oct 15, 2024
umesh3034 pushed a commit to umesh3034/cnats that referenced this issue Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proposal Enhancement idea or proposal
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant