- Oct 23, 2021
-
-
Stephen Belanger authored
PR-URL: https://github.com/nodejs/node/pull/40520 Reviewed-By:
Richard Lau <rlau@redhat.com> Reviewed-By:
Tobias Nießen <tniessen@tnie.de> Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com> Reviewed-By:
Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By:
Michael Dawson <midawson@redhat.com> Reviewed-By:
James M Snell <jasnell@gmail.com>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/40563 Reviewed-By:
Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Voltrex <mohammadkeyvanzade94@gmail.com>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/40527 Reviewed-By:
Luigi Pinca <luigipinca@gmail.com> Reviewed-By:
Antoine du Hamel <duhamelantoine1995@gmail.com>
-
voltrexmaster authored
Fixes: https://github.com/nodejs/node/issues/40323 PR-URL: https://github.com/nodejs/node/pull/40566 Reviewed-By:
Rich Trott <rtrott@gmail.com> Reviewed-By:
Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By:
Colin Ihrig <cjihrig@gmail.com>
-
- Oct 22, 2021
-
-
Rich Trott authored
Document the things that are considered when making the determination as to whether something should or shouldn't be in core. This does not (yet, at least) attempt to address *how* to include modules in core. (Should it be in the Node.js code base or vendored in from a separate repository?) It is limited to *whether* something should be in core or not. Closes: https://github.com/nodejs/TSC/issues/1041 PR-URL: https://github.com/nodejs/node/pull/40338 Fixes: https://github.com/nodejs/TSC/issues/1041 Reviewed-By:
Robert Nagy <ronagy@icloud.com> Reviewed-By:
Michael Dawson <midawson@redhat.com> Reviewed-By:
Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By:
Michaël Zasso <targos@protonmail.com>
-
Darshan Sen authored
Signed-off-by:
Darshan Sen <darshan.sen@postman.com> PR-URL: https://github.com/nodejs/node/pull/40457 Reviewed-By:
Anna Henningsen <anna@addaleax.net> Reviewed-By:
Richard Lau <rlau@redhat.com> Reviewed-By:
Tobias Nießen <tniessen@tnie.de>
-
Lu Yahan authored
PR-URL: https://github.com/nodejs/node/pull/40473 Reviewed-By:
Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By:
Richard Lau <rlau@redhat.com>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/40539 Reviewed-By:
Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By:
Richard Lau <rlau@redhat.com> Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Tobias Nießen <tniessen@tnie.de> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com>
-
Node.js GitHub Bot authored
PR-URL: https://github.com/nodejs/node/pull/40464 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Rich Trott <rtrott@gmail.com> Reviewed-By:
Michael Dawson <midawson@redhat.com> Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By:
Danielle Adams <adamzdanielle@gmail.com> Reviewed-By:
Richard Lau <rlau@redhat.com> Reviewed-By:
Сковорода Никита Андреевич <chalkerx@gmail.com> Reviewed-By:
Beth Griggs <bgriggs@redhat.com> Reviewed-By:
Myles Borins <myles.borins@gmail.com> Reviewed-By:
Franziska Hinkelmann <franziska.hinkelmann@gmail.com> Reviewed-By:
Yihong Wang <yh.wang@ibm.com>
-
- Oct 21, 2021
-
-
Francesco Trotta authored
Fixes #40305 PR-URL: https://github.com/nodejs/node/pull/40325 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By:
Guy Bedford <guybedford@gmail.com> Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Zeyu Yang <himself65@outlook.com>
-
Daniel Bevenius authored
PR-URL: https://github.com/nodejs/node/pull/40518 Refs: https://github.com/nodejs/node/issues/40509 Reviewed-By:
Richard Lau <rlau@redhat.com> Reviewed-By:
Beth Griggs <bgriggs@redhat.com>
-
Daniel Bevenius authored
PR-URL: https://github.com/nodejs/node/pull/40518 Refs: https://github.com/nodejs/node/issues/40509 Reviewed-By:
Richard Lau <rlau@redhat.com> Reviewed-By:
Beth Griggs <bgriggs@redhat.com>
-
Daniel Bevenius authored
Currently using the --openssl-is-fips configuration option in combination with --ninja is broken. This commit fixes two issues, one being an issue with the linker/version script path variable. The second is that the locations of built artifacts that differ for ninja and make. ninja: $ ./configure --openssl-is-fips --ninja $ ninja -C out/Release $ ./node --enable-fips -p 'crypto.getFips()' 1 make: $ ./configure --openssl-is-fips $ make -j8 $ ./node --enable-fips -p 'crypto.getFips()' 1 PR-URL: https://github.com/nodejs/node/pull/40518 Refs: https://github.com/nodejs/node/issues/40509 Reviewed-By:
Richard Lau <rlau@redhat.com> Reviewed-By:
Beth Griggs <bgriggs@redhat.com>
-
Michaël Zasso authored
Notable changes: Fixed distribution for native addon builds This release fixes an issue introduced in Node.js v17.0.0, where some V8 headers were missing from the distributed tarball, making it impossible to build native addons. These headers are now included. https://github.com/nodejs/node/pull/40526 Fixed stream issues * Fixed a regression in `stream.promises.pipeline`, which was introduced in version 16.10.0, is fixed. It is now possible again to pass an array of streams to the function. https://github.com/nodejs/node/pull/40193 * Fixed a bug in `stream.Duplex.from`, which didn't work properly when an async generator function was passed to it. https://github.com/nodejs/node/pull/40499 PR-URL: https://github.com/nodejs/node/pull/40535
-
Michael Dawson authored
Fix up example and make it more explicit on how you need to use napi_extended_error_info in order to help people avoid what might be a common mistake that we made in node-addon-api. Refs: https://github.com/nodejs/node-addon-api/issues/1089 Signed-off-by:
Michael Dawson <mdawson@devrus.com> PR-URL: https://github.com/nodejs/node/pull/40458 Reviewed-By:
Gabriel Schulhof <gabrielschulhof@gmail.com> Reviewed-By:
Darshan Sen <raisinten@gmail.com>
-
- Oct 20, 2021
-
-
Darshan Sen authored
There is no need to crash the process if any of these checks fail. Signed-off-by:
Darshan Sen <darshan.sen@postman.com> PR-URL: https://github.com/nodejs/node/pull/40425 Reviewed-By:
Anna Henningsen <anna@addaleax.net>
-
Richard Lau authored
Notable Changes: Experimental ESM Loader Hooks API: Node.js ESM Loader hooks have been consolidated to represent the steps involved needed to facilitate future loader chaining: 1. `resolve`: `resolve` [+ `getFormat`] 2. `load`: `getFormat` + `getSource` + `transformSource` For consistency, `getGlobalPreloadCode` has been renamed to `globalPreload`. A loader exporting obsolete hook(s) will trigger a single deprecation warning (per loader) listing the errant hooks. Contributed by Jacob Smith, Geoffrey Booth, and Bradley Farias - https://github.com/nodejs/node/pull/37468 Other Notable Changes: deps: * upgrade npm to 8.1.0 (npm team) https://github.com/nodejs/node/pull/40463 doc: * deprecate (doc-only) http abort related (dr-js) https://github.com/nodejs/node/pull/36670 vm: * (SEMVER-MINOR) add support for import assertions in dynamic imports (Antoine du Hamel) https://github.com/nodejs/node/pull/40249 PR-URL: https://github.com/nodejs/node/pull/40504
-
Michaël Zasso authored
Fixes: https://github.com/nodejs/Release/issues/704 PR-URL: https://github.com/nodejs/node/pull/40526 Fixes: https://github.com/nodejs/node/issues/40529 Reviewed-By:
Richard Lau <rlau@redhat.com> Reviewed-By:
Michael Dawson <midawson@redhat.com>
-
Mestery authored
PR-URL: https://github.com/nodejs/node/pull/40409 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Zijian Liu <lxxyxzj@gmail.com> Reviewed-By:
Qingyu Deng <i@ayase-lab.com>
-
Darshan Sen authored
Signed-off-by:
Darshan Sen <darshan.sen@postman.com> PR-URL: https://github.com/nodejs/node/pull/40400 Reviewed-By:
Anna Henningsen <anna@addaleax.net>
-
Darshan Sen authored
This merges the `IsEmpty()` call and the `ToLocalChecked()` call into a single `ToLocal()` call. Signed-off-by:
Darshan Sen <darshan.sen@postman.com> PR-URL: https://github.com/nodejs/node/pull/40386 Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Anna Henningsen <anna@addaleax.net>
-
Mestery authored
Fixes: https://github.com/nodejs/node/issues/40191 PR-URL: https://github.com/nodejs/node/pull/40193 Reviewed-By:
Luigi Pinca <luigipinca@gmail.com> Reviewed-By:
Robert Nagy <ronagy@icloud.com> Reviewed-By:
James M Snell <jasnell@gmail.com>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/40403 Reviewed-By:
Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By:
Zijian Liu <lxxyxzj@gmail.com> Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Antoine du Hamel <duhamelantoine1995@gmail.com>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/40403 Reviewed-By:
Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By:
Zijian Liu <lxxyxzj@gmail.com> Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Antoine du Hamel <duhamelantoine1995@gmail.com>
-
Richard Lau authored
Add a section to the Major Releases section of the release guide to cover updating the list of expected files for the promotion tool. PR-URL: https://github.com/nodejs/node/pull/40470 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Beth Griggs <bgriggs@redhat.com>
-
Michael Dawson authored
Coverity scan reported a free after use and I think its right. Tweak to avoid double free. Signed-off-by:
Michael Dawson <mdawson@devrus.com> PR-URL: https://github.com/nodejs/node/pull/40380 Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com> Reviewed-By:
Minwoo Jung <nodecorelab@gmail.com> Reviewed-By:
Darshan Sen <raisinten@gmail.com>
-
Rich Trott authored
Refs: https://github.com/nodejs/TSC/issues/1096 PR-URL: https://github.com/nodejs/node/pull/40523 Reviewed-By:
Richard Lau <rlau@redhat.com> Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/40517 Reviewed-By:
Tobias Nießen <tniessen@tnie.de> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com>
-
Luigi Pinca authored
The request uses chunked transfer encoding and the HTTP response status code is not 400 but 200. PR-URL: https://github.com/nodejs/node/pull/40486 Reviewed-By:
Tobias Nießen <tniessen@tnie.de> Reviewed-By:
Colin Ihrig <cjihrig@gmail.com>
-
Luigi Pinca authored
The `maxFreeSockets` option specifies the maximum number of free sockets per host. PR-URL: https://github.com/nodejs/node/pull/40483 Fixes: https://github.com/nodejs/node/issues/40446 Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Benjamin Gruenbaum <benjamingr@gmail.com>
-
- Oct 19, 2021
-
-
Beth Griggs authored
Notable Changes: Deprecations and Removals: - (SEMVER-MAJOR) dns: runtime deprecate type coercion of `dns.lookup` options (Antoine du Hamel) [https://github.com/nodejs/node/pull/39793] - doc: deprecate (doc-only) http abort related (dr-js) [https://github.com/nodejs/node/pull/36670] - (SEMVER-MAJOR) module: subpath folder mappings EOL (Guy Bedford) [https://github.com/nodejs/node/pull/40121] - (SEMVER-MAJOR) module: runtime deprecate trailing slash patterns (Guy Bedford) [https://github.com/nodejs/node/pull/40117] OpenSSL 3.0: Node.js now includes OpenSSL 3.0, specifically https://github.com/quictls/openssl which provides QUIC support. While OpenSSL 3.0 APIs should be mostly compatible with those provided by OpenSSL 1.1.1, we do anticipate some ecosystem impact due to tightened restrictions on the allowed algorithms and key sizes. If you hit an `ERR_OSSL_EVP_UNSUPPORTED` error in your application with Node.js 17, it’s likely that your application or a...
-
simon-id authored
PR-URL: https://github.com/nodejs/node/pull/40433 Reviewed-By:
Vladimir de Turckheim <vlad2t@hotmail.com> Reviewed-By:
Stephen Belanger <admin@stephenbelanger.com> Reviewed-By:
Gerhard Stöbich <deb2001-github@yahoo.de> Reviewed-By:
Michael Dawson <midawson@redhat.com> Reviewed-By:
Bryan English <bryan@bryanenglish.com> Reviewed-By:
Zijian Liu <lxxyxzj@gmail.com>
-
simon-id authored
PR-URL: https://github.com/nodejs/node/pull/40433 Reviewed-By:
Vladimir de Turckheim <vlad2t@hotmail.com> Reviewed-By:
Stephen Belanger <admin@stephenbelanger.com> Reviewed-By:
Gerhard Stöbich <deb2001-github@yahoo.de> Reviewed-By:
Michael Dawson <midawson@redhat.com> Reviewed-By:
Bryan English <bryan@bryanenglish.com> Reviewed-By:
Zijian Liu <lxxyxzj@gmail.com>
-
Robert Nagy authored
Fixes: https://github.com/nodejs/node/issues/40497 PR-URL: https://github.com/nodejs/node/pull/40499 Reviewed-By:
Matteo Collina <matteo.collina@gmail.com> Reviewed-By:
Benjamin Gruenbaum <benjamingr@gmail.com>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/40490 Reviewed-By:
Luigi Pinca <luigipinca@gmail.com> Reviewed-By:
Tobias Nießen <tniessen@tnie.de> Reviewed-By:
Zijian Liu <lxxyxzj@gmail.com>
-
Joyee Cheung authored
PR-URL: https://github.com/nodejs/node/pull/40239 Refs: https://github.com/nodejs/node/pull/38905 Refs: https://github.com/nodejs/node/issues/37476 Reviewed-By:
Anna Henningsen <anna@addaleax.net> Reviewed-By:
Minwoo Jung <nodecorelab@gmail.com> Reviewed-By:
James M Snell <jasnell@gmail.com>
-
Joyee Cheung authored
PR-URL: https://github.com/nodejs/node/pull/40239 Refs: https://github.com/nodejs/node/pull/38905 Refs: https://github.com/nodejs/node/issues/37476 Reviewed-By:
Anna Henningsen <anna@addaleax.net> Reviewed-By:
Minwoo Jung <nodecorelab@gmail.com> Reviewed-By:
James M Snell <jasnell@gmail.com>
-
Joyee Cheung authored
Split test-crypto-dh.js so that it is less likely to timeout on less powerful bots. PR-URL: https://github.com/nodejs/node/pull/40451 Refs: https://github.com/nodejs/reliability/issues/86 Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Rich Trott <rtrott@gmail.com>
-
Michael Dawson authored
This missing initialization was reported by the coverity scans we are in the process of re-enabling. Signed-off-by:
Michael Dawson <mdawson@devrus.com> PR-URL: https://github.com/nodejs/node/pull/40379 Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Minwoo Jung <nodecorelab@gmail.com> Reviewed-By:
Darshan Sen <raisinten@gmail.com>
-
npm team authored
PR-URL: https://github.com/nodejs/node/pull/40463 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com> Reviewed-By:
Michael Dawson <midawson@redhat.com>
-