delta/node-new.git/test/parallel/test-process-versions.js, branch cleanup-getnodeversion.py

http: remove legacy parser

2019-09-20T18:18:35+00:00

Remove the legacy `http_parser` implementation as a dependency
and all code that uses it in favor of llhttp, given that the latter
has been the default for all of Node 12 with no outstanding issues.

PR-URL: https://github.com/nodejs/node/pull/29589
Reviewed-By: Colin Ihrig 
Reviewed-By: Matteo Collina 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
Reviewed-By: Jiawen Geng 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Minwoo Jung 
Reviewed-By: Trivikram Kamat

build: expose napi_build_version variable

2019-07-02T05:07:42+00:00

Expose `napi_build_version` to allow `node-gyp` to make it
available for building native addons.

Fixes: https://github.com/nodejs/node-gyp/issues/1745
Refs: https://github.com/nodejs/abi-stable-node/issues/371
PR-URL: https://github.com/nodejs/node/pull/27835
Reviewed-By: Ben Noordhuis 
Reviewed-By: Anna Henningsen 
Reviewed-By: Gabriel Schulhof 
Reviewed-By: Luigi Pinca 
Reviewed-By: Richard Lau 
Reviewed-By: Michael Dawson

zlib: add brotli support

2019-01-05T20:36:46+00:00

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

Co-authored-by: Hackzzila 

PR-URL: https://github.com/nodejs/node/pull/24938
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Matteo Collina 
Reviewed-By: Jan Krems 
Reviewed-By: James M Snell 
Reviewed-By: Myles Borins 
Reviewed-By: Ali Ijaz Sheikh 
Reviewed-By: Daniel Bevenius

http: make parser choice a runtime flag

2018-12-06T04:21:36+00:00

Add a `--http-parser=llhttp` vs `--http-parser=traditional`
command line switch, to make testing and comparing the new
llhttp-based implementation easier.

PR-URL: https://github.com/nodejs/node/pull/24739
Refs: https://github.com/nodejs/node/issues/24730
Reviewed-By: Joyee Cheung 
Reviewed-By: Colin Ihrig 
Reviewed-By: Fedor Indutny 
Reviewed-By: Gus Caplan 
Reviewed-By: Matheus Marchini 
Reviewed-By: Matteo Collina 
Reviewed-By: Ali Ijaz Sheikh

deps: introduce `llhttp`

2018-11-10T22:54:21+00:00

llhttp is modern, written in human-readable TypeScript, verifiable, and
is very easy to maintain.

See: https://github.com/indutny/llhttp

PR-URL: https://github.com/nodejs/node/pull/24059
Reviewed-By: Matteo Collina 
Reviewed-By: Refael Ackermann 
Reviewed-By: James M Snell 
Reviewed-By: Rod Vagg 
Reviewed-By: Colin Ihrig 
Reviewed-By: Gus Caplan 
Reviewed-By: Ujjwal Sharma 
Reviewed-By: Ben Noordhuis

test: fix regression when compiled with FIPS

2018-10-28T14:12:28+00:00

In commit bff53c5a9d6, a check was added for very specific OpenSSL
format. Unfortunately, when OpenSSL is compiled in FIPS mode, this
check fails. Added additional regex to satisfy OpenSSL version
strings in both regular and FIPS modes.

PR-URL: https://github.com/nodejs/node/pull/23871
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig 
Reviewed-By: James M Snell 
Reviewed-By: Tobias Nießen 
Reviewed-By: Refael Ackermann 
Reviewed-By: Richard Lau 
Reviewed-By: Michael Dawson 
Reviewed-By: Sam Roberts 
Reviewed-By: Luigi Pinca

crypto: strip unwanted space from openssl version

2018-10-23T03:39:45+00:00

Remove trailing " \n" from `process.versions.openssl`.

d3d6cd3ecad19 was incorrectly printing this trailer, but because the
target buffer size was claimed to be the length of the version string,
the trailer was truncated off.

9ed4646df05b9 corrected the target buffer size, but then the trailer
started to appear in process.versions.

Added a test to check for regressions.

PR-URL: https://github.com/nodejs/node/pull/23678
Reviewed-By: Anna Henningsen 
Reviewed-By: James M Snell 
Reviewed-By: Refael Ackermann 
Reviewed-By: Richard Lau 
Reviewed-By: Colin Ihrig 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Sakthipriyan Vairamani 
Reviewed-By: Tobias Nießen 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Tiancheng "Timothy" Gu

n-api: expose n-api version in process.versions

2018-01-11T21:53:53+00:00

Expose n-api version in process.versions so that it is
available for use in javascript by external modules
like node-pre-gyp. It was previously accessible through
a functon available in the N-API.

PR-URL: https://github.com/nodejs/node/pull/18067
Reviewed-By: Anna Henningsen 
Reviewed-By: Tiancheng "Timothy" Gu 
Reviewed-By: James M Snell 
Reviewed-By: Jeremiah Senkpiel 
Reviewed-By: Colin Ihrig 
Reviewed-By: Minwoo Jung 
Reviewed-By: Gibson Fahnestock 
Reviewed-By: Tobias Nießn 
Reviewed-By: Richard Lau

build: add V8 embedder version string

2017-10-18T16:18:32+00:00

After this commit, `process.versions.v8` will look like:
"6.0.287.53-node.0".
The goal is that everytime we apply a non-official patch to `deps/v8`,
we increment our own number instead of V8's patch level.
This number must be set back to 0 after major V8 updates.

Fixes: https://github.com/nodejs/node/issues/15698
PR-URL: https://github.com/nodejs/node/pull/15785
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Gibson Fahnestock 
Reviewed-By: Ali Ijaz Sheikh 
Reviewed-By: James M Snell 
Reviewed-By: Michael Dawson

lib: simplify the readonly properties of icu

2017-08-30T21:08:49+00:00

Call Object.defineProperty() twice to set readonly property is
unnecessary.

PR-URL: https://github.com/nodejs/node/pull/13221
Reviewed-By: James M Snell 
Reviewed-By: Daijiro Wachi 
Reviewed-By: Ruben Bridgewater