Allow automatic pings to be sent on pubsub channel

[mdouglass]

Description

If you turn on automatic pings (via pingInterval) on a pubsub channel, you get the error: “Cannot send commands in PubSub mode”. PING is one of the legal commands on a pubsub channel, so we override the ping call to set ignorePubSubMode true and allow the ping to be sent on that channel.

We are unable to use the new pingInterval setting on clients being used for pubsub because of an incorrect error that PING is not allowed on pubsub channels.

---

Checklist

• Does npm test pass with this change (including linting)?
• Is the new or changed code fully tested?
• Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

[mdouglass]

fix for #2372

[leibale]

@mdouglass thanks for contributing! this issue was already fixed in #2344 (I removed the client-side validation for PubSub mode, so there is no need to use ignorePubSubMode anymore). This PR is not yet ready for a full release, but in the meantime you can use @redis/[email protected] (a "pre-release") which already includes a fix for this bug.

[mdouglass]

Thanks, I had looked around for something pre-existing and somehow missed that. :)
We use redis instead of @redis/client -- do you know the timeline for a new release?

[leibale]

redis is using @redis/client under the hood, you can override the @redis/client version to this one and it should work.
See this for updates.

[mdouglass]

Great, thanks!