{"action":"created","issue":{"url":"https://api.github.com/repos/nodejs/node/issues/7211","repository_url":"https://api.github.com/repos/nodejs/node","labels_url":"https://api.github.com/repos/nodejs/node/issues/7211/labels{/name}","comments_url":"https://api.github.com/repos/nodejs/node/issues/7211/comments","events_url":"https://api.github.com/repos/nodejs/node/issues/7211/events","html_url":"https://github.com/nodejs/node/pull/7211","id":159078506,"number":7211,"title":"http: fix no dumping after `maybeReadMore`","user":{"login":"indutny","id":238531,"avatar_url":"https://avatars.githubusercontent.com/u/238531?v=3","gravatar_id":"","url":"https://api.github.com/users/indutny","html_url":"https://github.com/indutny","followers_url":"https://api.github.com/users/indutny/followers","following_url":"https://api.github.com/users/indutny/following{/other_user}","gists_url":"https://api.github.com/users/indutny/gists{/gist_id}","starred_url":"https://api.github.com/users/indutny/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/indutny/subscriptions","organizations_url":"https://api.github.com/users/indutny/orgs","repos_url":"https://api.github.com/users/indutny/repos","events_url":"https://api.github.com/users/indutny/events{/privacy}","received_events_url":"https://api.github.com/users/indutny/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/nodejs/node/labels/http","name":"http","color":"c7def8"},{"url":"https://api.github.com/repos/nodejs/node/labels/lts-watch-v0.10","name":"lts-watch-v0.10","color":"e11d21"},{"url":"https://api.github.com/repos/nodejs/node/labels/lts-watch-v0.12","name":"lts-watch-v0.12","color":"e11d21"},{"url":"https://api.github.com/repos/nodejs/node/labels/lts-watch-v4.x","name":"lts-watch-v4.x","color":"eb6420"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":12,"created_at":"2016-06-08T04:54:48Z","updated_at":"2016-06-13T13:49:10Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nodejs/node/pulls/7211","html_url":"https://github.com/nodejs/node/pull/7211","diff_url":"https://github.com/nodejs/node/pull/7211.diff","patch_url":"https://github.com/nodejs/node/pull/7211.patch"},"body":"##### Checklist\r\n\r\n- [x] `make -j4 test` (UNIX) or `vcbuild test nosign` (Windows) passes\r\n- [x] a test and/or benchmark is included\r\n- [x] documentation is changed or added\r\n- [x] the commit message follows commit guidelines\r\n\r\n\r\n##### Affected core subsystem(s)\r\n\r\nhttp\r\n\r\n##### Description of change\r\n\r\nWhen `maybeReadMore` kicks in on a first bytes of incoming data, the\r\n`req.read(0)` will be invoked and the `req._consuming` will be set to\r\n`true`. This seemingly harmless property leads to a dire consequences:\r\nthe server won't call `req._dump()` and the whole HTTP/1.1 pipeline will\r\nhang (single connection)."},"comment":{"url":"https://api.github.com/repos/nodejs/node/issues/comments/225585950","html_url":"https://github.com/nodejs/node/pull/7211#issuecomment-225585950","issue_url":"https://api.github.com/repos/nodejs/node/issues/7211","id":225585950,"user":{"login":"mcollina","id":52195,"avatar_url":"https://avatars.githubusercontent.com/u/52195?v=3","gravatar_id":"","url":"https://api.github.com/users/mcollina","html_url":"https://github.com/mcollina","followers_url":"https://api.github.com/users/mcollina/followers","following_url":"https://api.github.com/users/mcollina/following{/other_user}","gists_url":"https://api.github.com/users/mcollina/gists{/gist_id}","starred_url":"https://api.github.com/users/mcollina/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mcollina/subscriptions","organizations_url":"https://api.github.com/users/mcollina/orgs","repos_url":"https://api.github.com/users/mcollina/repos","events_url":"https://api.github.com/users/mcollina/events{/privacy}","received_events_url":"https://api.github.com/users/mcollina/received_events","type":"User","site_admin":false},"created_at":"2016-06-13T13:49:10Z","updated_at":"2016-06-13T13:49:10Z","body":"LGTM, and I agree with @bnoordhuis."}}
{"actor":{"display_login":"mcollina"}}