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

Added support for (Burp) relative URIs #3

Merged
merged 2 commits into from Apr 29, 2013

Conversation

Projects
None yet
2 participants
@dnet
Contributor

dnet commented Apr 29, 2013

When using Burp Suite with oauth-proxy configured as an upstream proxy, it tends to send only relative URIs, so the proxy needs some help to figure out where to forward the request to.

I'm aware that this is not necessarily the fault of oauth-proxy but

  1. Burp Suite is not free nor open source software, so I have little chance to patch it and
  2. this shouldn't break existing use-cases of oauth-proxy.

While I was there, I also fixed an unnecessary list instantiation.

@mojodna

This comment has been minimized.

Show comment
Hide comment
@mojodna

mojodna Apr 29, 2013

Owner

Looks great! Thank you!

Owner

mojodna commented Apr 29, 2013

Looks great! Thank you!

mojodna added a commit that referenced this pull request Apr 29, 2013

Merge pull request #3 from silentsignal/master
Added support for (Burp) relative URIs by using a provided Host header.

@mojodna mojodna merged commit 7774e39 into mojodna:master Apr 29, 2013

@mojodna

This comment has been minimized.

Show comment
Hide comment
@mojodna

mojodna Apr 29, 2013

Owner

Published to PyPI as 1.0.4.

Owner

mojodna commented Apr 29, 2013

Published to PyPI as 1.0.4.

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