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 proxy headers handling #60

Merged

StefH merged 2 commits into WireMock-Net:master from Dreamescaper:proxy_fixes

Oct 28, 2017

Contributor

Dreamescaper commented Oct 28, 2017 •

edited by StefH

Loading

Fix following issues with proxy (and tests added):

content headers were not set in both request and response
cookies were not set in request
auto-redirect should not be performed when proxying requests
if redirect response happened, and absolute Location host is the one that we're proxying to, we should change it to called one
creating new HttpClient for each request in not the best idea. I changed it so it would be created only once per ResponseBuilder.

Dreamescaper and others added 2 commits

October 28, 2017 18:26


          Fix proxy headers handling

cbd5b2f


          Small refactor and moved proxy tests to new test-file.

9789b98

StefH approved these changes

View reviewed changes

StefH merged commit c8c9ab9 into WireMock-Net:master

Collaborator

StefH commented Oct 28, 2017

Thanks again!

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

Labels

None yet