Damien Diederen

Difficult to produce a reasonable test for this change, as it requires Kerberos and control over events. I was able to reproduce and verify the fix "interactively" with a hacked...

Hi @maoling, @eolivelli, @anmolnar, I have forward-ported @maoling's patch here: Here is the specific commit: (I have not opened another PR; this one is perfectly fine. @maoling, don't hesitate to...

Hi @shayhatsor, I have recently implemented SASL support for the @ewhauser client, used by some of our applications: https://github.com/ewhauser/zookeeper/pull/38 Your project seems to be more active; it would be great...

Hi @bitchkat, > I have a product in production that is using zookeepernetex. Good to know! > I plan on moving the server to Zookeeper 3.5 soon since the ZK...

Hi @shayhatsor, Thank you for your reply. > […] I actually built this client for https://github.com/dotnet/orleans. I used to be very active in that project. […] Microsoft project Orleans and...

Hi @smiley22, Any interest in this patch? Cheers, -D

> The commits were tested by me, it worked & fixed the DIGEST-MD5 authentication! thank you @ztzg Great! You're welcome.

Good catch, @yarthur1! It looks like this was introduced in [ZOOKEEPER-2311](https://issues.apache.org/jira/browse/ZOOKEEPER-2311). Cc: @eolivelli, @symat. Would you mind opening a new ticket, so that we can track this change with the...

Hi @tudor, @symat, Sorry for the delay; I've been a bit "disconnected" and almost missed this. I'll have a look ASAP. Cheers, -D

Hi @tudor, I’ve had a look at this; it's a nasty thing. `DeliverWatchersWrapper` in `tests/ZKMocks.cc` tries to stop (some) threads by forcefully setting `zh->close_requested=1` via `terminateZookeeperThreads`. `adaptor_finish` is not called...