Nordix / hiredis-cluster

C client library for Valkey/Redis Cluster. This project is used and sponsored by Ericsson. It is a fork of the now unmaintained hiredis-vip.
BSD 3-Clause "New" or "Revised" License
86 stars 42 forks source link

Invoke all callbacks during redisClusterAsyncDisconnect() #204

Closed bjosv closed 6 months ago

bjosv commented 6 months ago

Don't skip calling redisAsyncDisconnect(context) on contexts with errors. This makes sure that all pending request callbacks are invoked with a NULL reply, not only on contexts without errors.

Attempted to add a testcase covering this scenario, but could not reproduce it using simulated-redis tests.

Fixes #203