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

org.apache.jmeter.protocol.http.sampler.HTTPSampler not redirecting properly. #788

Closed
asfimport opened this issue Nov 27, 2001 · 3 comments

Comments

@asfimport
Copy link
Collaborator

Kris Young (Bug 5136):
This is really version 1.7Beta.

The redirectUrl(HttpURLConnection, URL, UrlConfig) method is not redirecting
properly.

When performing a GET with params, the params are appended to the redirected
location URL.

i.e. If we have:
http://my.server.com/servlet/Login?username=user&password=psswd

The login servlet redirects to:
http://my.server.com/servlet/Welcome?name=YourName

In the JMeter framework the login servlet redirects to:
http://my.server.com/servlet/Welcome?name=YourName?username=user&password=psswd

In my local copy of 'HTTPSampler' I added at line #383:
URL newUrl = new URL(loc);
---> urlConfig.removeArguments();
urlConfig.putProperty(UrlConfig.DOMAIN, newUrl.getHost());
urlConfig.putProperty(UrlConfig.PATH, newUrl.getFile());

This did fix the problem but I'm not sure where else this is needed.

Votes in Bugzilla: 1
Severity: major
OS: All

@asfimport
Copy link
Collaborator Author

Mike Stover (migrated from Bugzilla):
Implemented a fix very close to what is suggested here.

@asfimport
Copy link
Collaborator Author

Jordi Salvat i Alabart (migrated from Bugzilla):
Marking all bugs RESOLVED before JMeter 1.8's release date as VERIFIED.
Yes, it's pretty poor QA procedure, but there's bugs here lingering since JMeter
1.6, and we need to clean up a little.

@asfimport
Copy link
Collaborator Author

Jordi Salvat i Alabart (migrated from Bugzilla):
Bulk-closing all bugs RESOLVED before JMeter 1.8 release date.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant