{"action":"created","issue":{"url":"https://api.github.com/repos/request/request/issues/1881","labels_url":"https://api.github.com/repos/request/request/issues/1881/labels{/name}","comments_url":"https://api.github.com/repos/request/request/issues/1881/comments","events_url":"https://api.github.com/repos/request/request/issues/1881/events","html_url":"https://github.com/request/request/pull/1881","id":114466593,"number":1881,"title":"Fix tunneling after redirection from https","user":{"login":"falms","id":1049907,"avatar_url":"https://avatars.githubusercontent.com/u/1049907?v=3","gravatar_id":"","url":"https://api.github.com/users/falms","html_url":"https://github.com/falms","followers_url":"https://api.github.com/users/falms/followers","following_url":"https://api.github.com/users/falms/following{/other_user}","gists_url":"https://api.github.com/users/falms/gists{/gist_id}","starred_url":"https://api.github.com/users/falms/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/falms/subscriptions","organizations_url":"https://api.github.com/users/falms/orgs","repos_url":"https://api.github.com/users/falms/repos","events_url":"https://api.github.com/users/falms/events{/privacy}","received_events_url":"https://api.github.com/users/falms/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-11-01T12:08:00Z","updated_at":"2015-11-01T13:04:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/request/request/pulls/1881","html_url":"https://github.com/request/request/pull/1881","diff_url":"https://github.com/request/request/pull/1881.diff","patch_url":"https://github.com/request/request/pull/1881.patch"},"body":"Current stable module cannot handle https-to-http redirection with proxy properly.\r\n\r\nWhen redirected from https to **http** with `options.tunnel = undefined`,\r\nthe **http** request should be proxied without CONNECT tunneling.\r\n\r\nThis pull-request includes these changes:\r\n\r\n1. use `options` again after redirection.\r\n2. fix test case for https-to-http redirection.\r\n3. fix description about `tunnel` option.\r\n\r\nThis pull-request solves [this problem](https://github.com/atom/apm/issues/322)."},"comment":{"url":"https://api.github.com/repos/request/request/issues/comments/152824173","html_url":"https://github.com/request/request/pull/1881#issuecomment-152824173","issue_url":"https://api.github.com/repos/request/request/issues/1881","id":152824173,"user":{"login":"simov","id":1694112,"avatar_url":"https://avatars.githubusercontent.com/u/1694112?v=3","gravatar_id":"","url":"https://api.github.com/users/simov","html_url":"https://github.com/simov","followers_url":"https://api.github.com/users/simov/followers","following_url":"https://api.github.com/users/simov/following{/other_user}","gists_url":"https://api.github.com/users/simov/gists{/gist_id}","starred_url":"https://api.github.com/users/simov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/simov/subscriptions","organizations_url":"https://api.github.com/users/simov/orgs","repos_url":"https://api.github.com/users/simov/repos","events_url":"https://api.github.com/users/simov/events{/privacy}","received_events_url":"https://api.github.com/users/simov/received_events","type":"User","site_admin":false},"created_at":"2015-11-01T13:04:42Z","updated_at":"2015-11-01T13:04:42Z","body":"I don't feel comfortable altering that logic. Why do you think that the current implementation is wrong?\r\n\r\nI would like @nylen to chime in here as well."}}