delta/node-new.git/lib/http.js, branch docs-fix

http: tidy up exposure of header validation

2020-05-25T17:21:57+00:00

PR-URL: https://github.com/nodejs/node/pull/33371
Reviewed-By: Anna Henningsen 
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Luigi Pinca 
Reviewed-By: Ruben Bridgewater

http: expose http.validate-header-name/value

2020-05-09T05:55:55+00:00

The use-case is for any framework that provides user mw a response
replacement, that collects the desired response state, and applies them
only on conclusion. As such a framework, I'd want to validate the
header names and values as soon as the user-code provides them.
This - to eliminate errors on response-send time, and provide developer
stack trace that contains the line that submits the offending values.

PR-URL: https://github.com/nodejs/node/pull/33119
Reviewed-By: Anna Henningsen

lib: flatten access to primordials

2019-11-25T09:28:15+00:00

Store all primordials as properties of the primordials object.
Static functions are prefixed by the constructor's name and prototype
methods are prefixed by the constructor's name followed by "Prototype".
For example: primordials.Object.keys becomes primordials.ObjectKeys.

PR-URL: https://github.com/nodejs/node/pull/30610
Refs: https://github.com/nodejs/node/issues/29766
Reviewed-By: Anna Henningsen 
Reviewed-By: Matteo Collina 
Reviewed-By: Colin Ihrig 
Reviewed-By: Trivikram Kamat

lib: enforce the use of Object from primordials

2019-04-12T03:38:45+00:00

PR-URL: https://github.com/nodejs/node/pull/27146
Reviewed-By: Colin Ihrig 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Joyee Cheung 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Anna Henningsen 
Reviewed-By: James M Snell 
Reviewed-By: Gus Caplan

benchmark,lib: change var to const

2019-03-30T12:16:39+00:00

Refs: https://github.com/nodejs/node/pull/26679

PR-URL: https://github.com/nodejs/node/pull/26915
Reviewed-By: Michaël Zasso 
Reviewed-By: Tobias Nießen 
Reviewed-By: Refael Ackermann

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