search

awr / KafkaClient

Native C# client for Kafka
Apache License 2.0
5 stars 2 forks source link

Reconnect on SslTransport should retry if ssl authentication fails #24

Open awr opened 7 years ago

awr commented 7 years ago

This may require refactoring the ReconnectingSocket code so the SslTransport can participate in the retry cycle directly.

To close this, we should have tests to prove this is working when ssl auth fails -> succeeds.

ligu commented 7 years ago

I had a race condition problem during connecting to kafka endpoints, when trying the latest version from Your master branch. Here is a tested and working fix (I've just fixed the SslTransport):

https://github.com/ligu/KafkaClient/tree/connection_fix

Since I have no more time, and have to deliver a solution this week, most probably this version (together with #26 ) will go to production on our side, so will be heavily tested soon :)