Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Fix improper sample code in http.markdown #25471

Closed
wants to merge 1 commit into from
Closed

Fix improper sample code in http.markdown #25471

wants to merge 1 commit into from

Conversation

hidekiy
Copy link

@hidekiy hidekiy commented Jun 2, 2015

You must consume the data from the response object. #8443
This sample code hangs up after consuming http agent's slots.

You must consume the data from the response object. #8443
@jasnell
Copy link
Member

jasnell commented Jun 4, 2015

LGTM

jasnell pushed a commit that referenced this pull request Jun 5, 2015
You must consume the data from the response object. #8443

Reviewed-By: James M Snell <[email protected]>
PR-URL: #25471
@jasnell
Copy link
Member

jasnell commented Jun 5, 2015

Landed in 6671efa

@jasnell jasnell closed this Jun 5, 2015
jasnell pushed a commit to nodejs/node that referenced this pull request Dec 14, 2015
Refs: nodejs/node-v0.x-archive#25471
Refs: nodejs/node-v0.x-archive#8443
PR-URL: #4263
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: James M Snell <[email protected]>
cjihrig pushed a commit to nodejs/node that referenced this pull request Dec 15, 2015
Refs: nodejs/node-v0.x-archive#25471
Refs: nodejs/node-v0.x-archive#8443
PR-URL: #4263
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit to nodejs/node that referenced this pull request Dec 30, 2015
Refs: nodejs/node-v0.x-archive#25471
Refs: nodejs/node-v0.x-archive#8443
PR-URL: #4263
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit to nodejs/node that referenced this pull request Jan 19, 2016
Refs: nodejs/node-v0.x-archive#25471
Refs: nodejs/node-v0.x-archive#8443
PR-URL: #4263
Reviewed-By: Rod Vagg <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: James M Snell <[email protected]>
scovetta pushed a commit to scovetta/node that referenced this pull request Apr 2, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants