Skip to content

Pub-sub connections use unmapped addresses after Sentinel failover #3192

Description

@elimelt

Description

I recently added nodeAddressMap support, but only after running integration tests that simulate a failure during pub-sub did I realize the address the connection is moved to doesn't use the address mappings.

Confirmed #3190 fixes the issue by patching it in my own codebase. If I must I'll write a test for it, but that will require refactoring some existing tests so as not to completely duplicate the scenario with only one change

Node.js Version

v22.17.0

Redis Server Version

7.4.8

Node Redis Version

5.11.0

Platform

Linux

Logs

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions