Connection string port is parsed wrong

EventStore / EventStore-Client-Dotnet

Dotnet Client SDK for the Event Store gRPC Client API written in C#

Other

140 stars 38 forks source link

Connection string port is parsed wrong #260

Closed xperiandri closed 6 months ago

xperiandri commented 1 year ago

Describe the bug Parsing a connection string esdb+discover://admin:changeit@localhost:52113?tls=false&keepAliveTimeout=10000&keepAliveInterval=10000 does not read port

To Reproduce Parse the connection string above

Expected behavior EventStoreClientConnectivitySettings.Address = http://localhost:52113/

Actual behavior EventStoreClientConnectivitySettings.Address = http://localhost:2113/

EventStore details

EventStore client version: 23.0.0.0

_DEV-95

timothycoleman commented 11 months ago

hi @xperiandri

i think because it is using edsb+discover (and not just esdb) the connection info ends up in a different part of the settings. instead of in Address is goes in DnsGossipSeeds.

Is the client attempting to connect to the wrong port (hopefully not)?

xperiandri commented 11 months ago

It is not

timothycoleman commented 11 months ago

Am I right in thinking that this is resolved?