{"action":"created","issue":{"url":"https://api.github.com/repos/getlantern/http-proxy/issues/71","labels_url":"https://api.github.com/repos/getlantern/http-proxy/issues/71/labels{/name}","comments_url":"https://api.github.com/repos/getlantern/http-proxy/issues/71/comments","events_url":"https://api.github.com/repos/getlantern/http-proxy/issues/71/events","html_url":"https://github.com/getlantern/http-proxy/pull/71","id":120691729,"number":71,"title":"[don't merge] fix #70","user":{"login":"fffw","id":10410437,"avatar_url":"https://avatars.githubusercontent.com/u/10410437?v=3","gravatar_id":"","url":"https://api.github.com/users/fffw","html_url":"https://github.com/fffw","followers_url":"https://api.github.com/users/fffw/followers","following_url":"https://api.github.com/users/fffw/following{/other_user}","gists_url":"https://api.github.com/users/fffw/gists{/gist_id}","starred_url":"https://api.github.com/users/fffw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fffw/subscriptions","organizations_url":"https://api.github.com/users/fffw/orgs","repos_url":"https://api.github.com/users/fffw/repos","events_url":"https://api.github.com/users/fffw/events{/privacy}","received_events_url":"https://api.github.com/users/fffw/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-12-07T03:35:20Z","updated_at":"2015-12-07T08:42:56Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/getlantern/http-proxy/pulls/71","html_url":"https://github.com/getlantern/http-proxy/pull/71","diff_url":"https://github.com/getlantern/http-proxy/pull/71.diff","patch_url":"https://github.com/getlantern/http-proxy/pull/71.patch"},"body":"See discussions around relevant code."},"comment":{"url":"https://api.github.com/repos/getlantern/http-proxy/issues/comments/162449238","html_url":"https://github.com/getlantern/http-proxy/pull/71#issuecomment-162449238","issue_url":"https://api.github.com/repos/getlantern/http-proxy/issues/71","id":162449238,"user":{"login":"uaalto","id":12701448,"avatar_url":"https://avatars.githubusercontent.com/u/12701448?v=3","gravatar_id":"","url":"https://api.github.com/users/uaalto","html_url":"https://github.com/uaalto","followers_url":"https://api.github.com/users/uaalto/followers","following_url":"https://api.github.com/users/uaalto/following{/other_user}","gists_url":"https://api.github.com/users/uaalto/gists{/gist_id}","starred_url":"https://api.github.com/users/uaalto/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/uaalto/subscriptions","organizations_url":"https://api.github.com/users/uaalto/orgs","repos_url":"https://api.github.com/users/uaalto/repos","events_url":"https://api.github.com/users/uaalto/events{/privacy}","received_events_url":"https://api.github.com/users/uaalto/received_events","type":"User","site_admin":false},"created_at":"2015-12-07T08:42:56Z","updated_at":"2015-12-07T08:42:56Z","body":"The reason is because requests to a proxy differ from normal requests in that the complete URL of the resource is being requested instead of just the path (as you are probably aware). By overwriting Opaque, the expected behavior is to remove the domain when available, since Path represents the path part of the URI. Failure to do so will break some web servers. If the provided solution causes an issue with encoding, then we might have to do it manually, but it surprises me. Path is not supposed to be altered, and your solution should be fine. What's the problem with it?"}}