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 followredirect when a redirect has a query string. #813

Closed
wants to merge 1 commit into from

Conversation

clitnak
Copy link

@clitnak clitnak commented Nov 9, 2012

When following redirects and the redirect has a query string, gatling escapes the question mark to %3F and makes all redirects of this type return a 404.

This pull has the fix for this issue.

@nremond
Copy link
Contributor

nremond commented Nov 9, 2012

Is this the same issue as this one ? #615

I would really appreciate being able to write a test case associated to this point. Do you have an exemple of a URL which would not work ?

thanks

@slandelle
Copy link
Member

I agree, query params are not properly handled when Location is relative.
Will fix, but in a different way.

Thanks for reporting.

@slandelle slandelle closed this Nov 9, 2012
slandelle pushed a commit that referenced this pull request Nov 9, 2012
@ghost ghost assigned slandelle Nov 9, 2012
slandelle pushed a commit that referenced this pull request Nov 19, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

4 participants