Skip to content
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

Fix redirect routing on request with no route. #870

Merged
merged 1 commit into from
Jun 21, 2023

Conversation

nihohit
Copy link
Contributor

@nihohit nihohit commented Jun 20, 2023

If a request's route field is set to None and the request was redirected, it should go to the redirected address regardless of the original route. Currently it goes to a random node.

If a request's `route` field is set to None and the request was
redirected, it should go to the redirected address regardless of the
original route. Currently it goes to a random node.
@nihohit nihohit changed the title Fix redirect routing on request with no route. Fix redirect routing on redirect with no route. Jun 20, 2023
@nihohit nihohit changed the title Fix redirect routing on redirect with no route. Fix redirect routing on request with no route. Jun 20, 2023
@jaymell
Copy link
Contributor

jaymell commented Jun 21, 2023

Thanks -- looks good to me!

@jaymell jaymell merged commit 5d9a491 into redis-rs:main Jun 21, 2023
9 checks passed
altanozlu pushed a commit to altanozlu/redis-rs that referenced this pull request Aug 16, 2023
If a request's `route` field is set to None and the request was
redirected, it should go to the redirected address regardless of the
original route. Currently it goes to a random node.
@shachlanAmazon shachlanAmazon deleted the fix-order branch November 15, 2023 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants