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.

Sign up for GitHub

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

Jump to bottom

fix redirect url should include rawquery #363

Merged
merged 1 commit into from
Dec 6, 2013
Merged

fix redirect url should include rawquery #363

merged 1 commit into from
Dec 6, 2013

Conversation

xiang90
Copy link
Contributor

@xiang90 xiang90 commented Dec 3, 2013

fix #362

@bcwaldon
Copy link
Contributor

bcwaldon commented Dec 3, 2013

Fix makes sense. I'd like @polvi to verify this fixes the problem we were seeing before it lands

@philips
Copy link
Contributor

philips commented Dec 4, 2013

lgtm too

@philips
Copy link
Contributor

philips commented Dec 4, 2013

Confirmed this fix works with etcdctl. We can merge as is but I would like a test. I will look into adding one now.

xiang90 added a commit that referenced this pull request Dec 6, 2013
@xiang90
Merge pull request #363 from xiangli-cmu/fix_consistent_redirection
5edaee7 
fix redirect url should include rawquery
@xiang90 xiang90 merged commit 5edaee7 into etcd-io:master Dec 6, 2013
@xiang90 xiang90 deleted the fix_consistent_redirection branch December 6, 2013 04:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

consistent seems broken with waitIndex
3 participants
@xiang90 @bcwaldon @philips