Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
blackmad opened this issue Apr 15, 2021 · 1 comment
Closed

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

blackmad opened this issue Apr 15, 2021 · 1 comment

Comments

@blackmad
Copy link

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 redis/node-redis#1157

@mmkal
Copy link
Owner

mmkal commented Jun 10, 2021

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.

@mmkal mmkal closed this as completed Jun 10, 2021
Repository owner locked and limited conversation to collaborators Jun 10, 2021

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants