search

mmkal / handy-redis

A wrapper around node_redis with Promise and TypeScript support.
Apache License 2.0
172 stars 10 forks source link

"XXXX can't be processed. The connection is already closed." #283

Closed blackmad closed 3 years ago

blackmad commented 3 years ago

I've noticed a bunch of these errors, would it be helpful if all the handy-redis wrappers checked .nodeClient.connected before attempting commands?

cf https://github.com/NodeRedis/node-redis/issues/1157

mmkal commented 3 years ago

What would you suggest this library do here? In general, I would prefer to keep this is as a thin wrapper but if you can propose something specific that doesn't add a lot of responsibility to this package, I'd consider it.