delta/node-new.git/lib/http.js, branch pull-request-doc-updates

lib: support overriding http\s.globalAgent

2019-01-07T23:10:36+00:00

Overriding `require('http[s]').globalAgent` is now respected by
consequent requests.

In order to achieve that, the following changes were made:

1. Implmentation in `http`: `module.exports.globalAgent` is now defined
through `Object.defineProperty`. Its getter and setter return \ set
`require('_http_agent').globalAgent`.

2. Implementation in `https`: the https `globalAgent` is not the same
as `_http_agent`, and is defined in `https` module itself. Therefore,
the fix here was to simply use `module.exports.globalAgent` to support
mutation.

3. According tests were added for both `http` and `https`, where in
both we create a server, set the default agent to a newly created
instance and make a request to that server. We then assert that the
given instance was actually used by inspecting its sockets property.

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

PR-URL: https://github.com/nodejs/node/pull/25170
Reviewed-By: James M Snell

http: add maxHeaderSize property

2018-12-20T18:15:07+00:00

This commit exposes the value of --max-http-header-size
as a property of the http module.

PR-URL: https://github.com/nodejs/node/pull/24860
Reviewed-By: Richard Lau 
Reviewed-By: Matteo Collina 
Reviewed-By: Michael Dawson 
Reviewed-By: Shelley Vohr 
Reviewed-By: James M Snell

http: allow url and options to be passed to http.request and http.get

2018-07-13T13:09:39+00:00

Fixes: https://github.com/nodejs/node/issues/20795
PR-URL: https://github.com/nodejs/node/pull/21616
Reviewed-By: Tiancheng "Timothy" Gu 
Reviewed-By: Ron Korving 
Reviewed-By: James M Snell 
Reviewed-By: Matteo Collina

http: support server options on createServer

2018-03-25T09:51:32+00:00

PR-URL: https://github.com/nodejs/node/pull/19461
Reviewed-By: Anna Henningsen 
Reviewed-By: Evan Lucas 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell

http: use more destructuring

2018-03-22T15:23:06+00:00

PR-URL: https://github.com/nodejs/node/pull/19481
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig 
Reviewed-By: Jackson Tian 
Reviewed-By: Luigi Pinca

lib: use destructuring for some constants

2017-10-16T21:34:32+00:00

This change is to unify the declaration for constants into using
destructuring on the top-level-module scope, reducing some redundant
code.

PR-URL: https://github.com/nodejs/node/pull/16063
Reviewed-By: James M Snell 
Reviewed-By: Anna Henningsen 
Reviewed-By: Tobias Nießen 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Gibson Fahnestock

http: use more efficient module.exports pattern

2017-03-20T23:01:31+00:00

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

meta: restore original copyright header

2017-03-10T19:23:48+00:00

A prior io.js era commit inappropriately removed the
original copyright statements from the source. This
restores those in any files still remaining from that
edit.

Ref: https://github.com/nodejs/TSC/issues/174
Ref: https://github.com/nodejs/node/pull/10599
PR-URL: https://github.com/nodejs/node/pull/10155

Note: This PR was required, reviewed-by and approved
by the Node.js Foundation Legal Committee and the TSC.
There is no `Approved-By:` meta data.

http: name anonymous functions in http

2016-10-18T22:38:43+00:00

Refs: https://github.com/nodejs/node/issues/8913
PR-URL: https://github.com/nodejs/node/pull/9055
Reviewed-By: James M Snell 
Reviewed-By: Luigi Pinca 
Reviewed-By: Franziska Hinkelmann 
Reviewed-By: Michaël Zasso

http: remove deprecated Client interface

2016-08-30T01:44:58+00:00

PR-URL: https://github.com/nodejs/node/pull/8104
Reviewed-By: James M Snell 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Colin Ihrig