fix(cluster): prevent infinite loop by nkaradzhov · Pull Request #3078 · redis/node-redis
Navigation Menu
{{ message }}
redis / node-redis Public
- Notifications You must be signed in to change notification settings
- Fork 1.9k
Merged
nkaradzhov merged 1 commit intoredis:masterfrom
Sep 12, 2025Merged
fix(cluster): prevent infinite loop#3078
nkaradzhov merged 1 commit intoredis:masterfrom
fix(cluster): prevent infinite loop#3078
nkaradzhov merged 1 commit intoredis:masterfrom
Conversation
Copy link Copy Markdown
Collaborator
nkaradzhov
commented
Sep 10, 2025
nkaradzhov
commented
getRandomNode could end up in an infinite loop if
this.masters is empty and this.replicas is empty.
fixes: #3075
- Is the new or changed code fully tested?
getRandomNode could end up in an infinite loop if this.masters is empty and this.replicas is empty. fixes: redis#3075
madhusudancs approved these changes Sep 10, 2025
PavelPashov approved these changes Sep 11, 2025
nkaradzhov
deleted the
cluster-slots-fix
branch
chrisbbreuer
mentioned this pull request
1 task
This was referenced
Nov 19, 2025This was referenced
Feb 8, 2026This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants