Skip to content

fix(microservices): correct pub/sub client type in redis listeners - #17331

Merged
kamilmysliwiec merged 1 commit into
nestjs:masterfrom
zaewc:fix/redis-server-swapped-client-type
Jul 20, 2026
Merged

fix(microservices): correct pub/sub client type in redis listeners#17331
kamilmysliwiec merged 1 commit into
nestjs:masterfrom
zaewc:fix/redis-server-swapped-client-type

Conversation

@zaewc

@zaewc zaewc commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

What is the current behavior?

Issue Number: #17330

What is the new behavior?

Index 0 is labeled 'sub' and index 1 is labeled'pub', matching on() and ClientRedis. Added a regression test that fails without the fix

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 917

Coverage decreased (-0.003%) to 90.297%

Details

  • Coverage decreased (-0.003%) from the base build.
  • Patch coverage: 1 of 1 lines across 1 file are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 8657
Covered Lines: 7817
Line Coverage: 90.3%
Relevant Branches: 2949
Covered Branches: 2404
Branch Coverage: 81.52%
Branches in Coverage %: No
Coverage Strength: 56.63 hits per line

💛 - Coveralls

@kamilmysliwiec
kamilmysliwiec merged commit 195f4af into nestjs:master Jul 20, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants