Skip to content

Fix proxy with TLS scenario with SocketConnectionFactory#826

Merged
mtmk merged 3 commits into
nats-io:mainfrom
wolfkor:fix-802
Apr 15, 2025
Merged

Fix proxy with TLS scenario with SocketConnectionFactory#826
mtmk merged 3 commits into
nats-io:mainfrom
wolfkor:fix-802

Conversation

@wolfkor

@wolfkor wolfkor commented Apr 13, 2025

Copy link
Copy Markdown
Contributor

@mtmk mtmk changed the title fix proxy with TLS scenario ba SocketConnectionFactory (#802) fix proxy with TLS scenario with SocketConnectionFactory Apr 13, 2025

@mtmk mtmk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM thanks @wolfkor

@mtmk
mtmk requested a review from caleblloyd April 13, 2025 18:50
@mtmk mtmk changed the title fix proxy with TLS scenario with SocketConnectionFactory Fix proxy with TLS scenario with SocketConnectionFactory Apr 13, 2025
Comment thread src/NATS.Client.Core/NatsOpts.cs Outdated
@mtmk
mtmk merged commit 30598aa into nats-io:main Apr 15, 2025
mtmk added a commit that referenced this pull request Apr 15, 2025
mtmk added a commit that referenced this pull request Apr 15, 2025
mtmk pushed a commit that referenced this pull request Apr 15, 2025
* fix proxy with TLS scenario ba SocketConnectionFactory (#802) (#804)

* init (#826)

* socket connection factory

Signed-off-by: Caleb Lloyd <caleblloyd@gmail.com>

* standardize ConnectAsync

Signed-off-by: Caleb Lloyd <caleblloyd@gmail.com>

* clean up INatsSocketConnection

Signed-off-by: Caleb Lloyd <caleblloyd@gmail.com>

* improve doc

Signed-off-by: Caleb Lloyd <caleblloyd@gmail.com>

---------

Signed-off-by: Caleb Lloyd <caleblloyd@gmail.com>
Co-authored-by: wolfkor <w.kordts@mid.de>
@wolfkor
wolfkor deleted the fix-802 branch April 15, 2025 11:49
mtmk added a commit that referenced this pull request Apr 16, 2025
* Stopped testing with net6.0 (#821)
* Update JetStream models (#832)
* Add direct request-reply (#829)
* Socket Connection Factory (#828)
* Fix proxy with TLS scenario with SocketConnectionFactory (#826)
* Make NatsJSConstants public (#825)
* Use Stopwatch in PingCommand for more precise timing (#827)
* Fix netstandard bugs and tidy up tests (#817)
@mtmk mtmk mentioned this pull request Apr 16, 2025
mtmk added a commit that referenced this pull request Apr 16, 2025
* Stopped testing with net6.0 (#821)
* Update JetStream models (#832)
* Add direct request-reply (#829)
* Socket Connection Factory (#828)
* Fix proxy with TLS scenario with SocketConnectionFactory (#826)
* Make NatsJSConstants public (#825)
* Use Stopwatch in PingCommand for more precise timing (#827)
* Fix netstandard bugs and tidy up tests (#817)
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.

Communication with TLS bus behind proxy results in SslPolicyErrors.RemoteCertificateNameMismatch

2 participants