80dB / AsyncNats

An async Nats.io client written using C# 8.0 language options
MIT License
6 stars 2 forks source link

add nats server pool and dns resolver #9

Closed israellot closed 2 years ago

israellot commented 2 years ago

Hey @80dB , This adds a server pool and dns as discussed.

Small addition: also removes that ugly PublishRaw from the public api by making it internal but visible to the benchmark project.

israellot commented 2 years ago

Please check the updated PR.

80dB commented 2 years ago

Different side note, we have the option to "add discovered servers" - shouldn't we have the ability to only use discovered servers? (instead of adding)

israellot commented 2 years ago

Ok, I've got your point @80dB. I was a bit short on time, but please take a look at the updated PR now. Added the ability to specify a custom implementation of a server pool, and at the default one, it first rotates ips for a single dns before moving on to the next host name.

israellot commented 2 years ago

Just updated again.

israellot commented 2 years ago

@80dB I guess it needs merging