delta/node-new.git/lib/_http_client.js, branch Ninja.py-StringIO

http: simplify timeout handling

2019-08-31T16:41:41+00:00

Avoids allocating and registering extra listeners for 'timeout'.

PR-URL: https://github.com/nodejs/node/pull/29200
Reviewed-By: Luigi Pinca 
Reviewed-By: Ruben Bridgewater

http: fix event listener leak

2019-08-23T13:28:36+00:00

Fixes: https://github.com/nodejs/node/issues/29239
PR-URL: https://github.com/nodejs/node/pull/29245
Reviewed-By: James M Snell 
Reviewed-By: Luigi Pinca 
Reviewed-By: Jiawen Geng 
Reviewed-By: Matteo Collina

http: simplify drain()

2019-08-19T19:56:08+00:00

Simplify and slightly optimize draining outgoing http streams. Avoid
extra event listener and inline with rest of the drain logic.

PR-URL: https://github.com/nodejs/node/pull/29081
Reviewed-By: Matteo Collina

http: replace superfluous connection property with getter/setter

2019-08-17T13:21:59+00:00

PR-URL: https://github.com/nodejs/node/pull/29015
Reviewed-By: Anna Henningsen 
Reviewed-By: James M Snell 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Luigi Pinca 
Reviewed-By: Rich Trott

http: follow symbol naming convention

2019-08-14T22:54:01+00:00

PR-URL: https://github.com/nodejs/node/pull/29091
Reviewed-By: Luigi Pinca 
Reviewed-By: Anna Henningsen 
Reviewed-By: James M Snell 
Reviewed-By: Rich Trott

http: avoid extra listener

2019-07-19T03:58:54+00:00

PR-URL: https://github.com/nodejs/node/pull/28705
Reviewed-By: James M Snell 
Reviewed-By: Rich Trott 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Fedor Indutny

http: expose headers on an http.ClientRequest "information" event

2019-07-16T04:22:11+00:00

1xx intermediate status responses are allowed to have headers; so
expose the "httpVersion", "httpVersionMajor", "httpVersionMinor",
"headers", "rawHeaders", and "statusMessage" properties on this
event.

PR-URL: https://github.com/nodejs/node/pull/28459
Reviewed-By: Luigi Pinca 
Reviewed-By: Matteo Collina 
Reviewed-By: Rich Trott

Revert "http: fix test where aborted should not be emitted"

2019-07-15T15:40:19+00:00

This reverts commit 461bf36d701f3f7c669e2d916d5a5bc17fc447bf.

PR-URL: https://github.com/nodejs/node/pull/28699
Reviewed-By: Colin Ihrig 
Reviewed-By: Matteo Collina

http: fix test where aborted should not be emitted

2019-07-15T04:48:21+00:00

PR-URL: https://github.com/nodejs/node/pull/20077
Fixes: https://github.com/nodejs/node/issues/20107
Reviewed-By: James M Snell 
Reviewed-By: Matteo Collina

http: improve parser error messages

2019-07-06T05:36:53+00:00

Include the library-provided reason in the Error’s `message`.

Fixes: https://github.com/nodejs/node/issues/28468

PR-URL: https://github.com/nodejs/node/pull/28487
Reviewed-By: Anto Aravinth 
Reviewed-By: Daniel Bevenius 
Reviewed-By: Fedor Indutny 
Reviewed-By: Rich Trott 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Colin Ihrig 
Reviewed-By: Trivikram Kamat