summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* tools: update all dependencies of markdown linterMichaël Zasso2021-09-223-8179/+1785
| | | | | | | | PR-URL: https://github.com/nodejs/node/pull/40035 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
* deps: patch v8 for vs2019 in std17Jiawen Geng2021-09-223-3/+4
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/40060 Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
* worker: avoid potential deadlock on NearHeapLimitSantiago Gimeno2021-09-223-5/+37
| | | | | | | | | | | | | | | | | | | | | It can happen that the `NearHeapLimit` callback is called while calling the `oninit()` function on `MessagePort` construction causing a deadlock when the `Worker::Exit()` method is called, as the `mutex_` was already held on the `CreateEnvMessagePort()` method. To fix it, just use the `mutex_` to protect the `child_port_data_` variable and avoid holding it when creating the `MessagePort`. Also, return early from `Worker::Run()` if the worker message port could not be created. Fixes: https://github.com/nodejs/node/issues/38208 PR-URL: https://github.com/nodejs/node/pull/38403 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
* test: fix internet/test-dnsRich Trott2021-09-221-1/+1
| | | | | | | | | | internet/test-dns is failing due to a typo that inadvertently sends a boolean instead of a regular expression. PR-URL: https://github.com/nodejs/node/pull/40083 Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Qingyu Deng <i@ayase-lab.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de>
* meta: consolidate AUTHORS entry for mikemaccanaRich Trott2021-09-222-2/+2
| | | | | | | | PR-URL: https://github.com/nodejs/node/pull/40051 Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de>
* doc: add Ayase-252 to collaboratorsQingyu Deng2021-09-221-0/+2
| | | | | | | | Fixes: https://github.com/nodejs/node/issues/39912 PR-URL: https://github.com/nodejs/node/pull/40078 Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
* src: fix -Wunreachable-code-return errorShelley Vohr2021-09-222-2/+0
| | | | | | | | | PR-URL: https://github.com/nodejs/node/pull/40034 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net>
* doc: fix CCM cipher example in MJSTobias Nießen2021-09-221-4/+2
| | | | | | | | | | | | | The original example used 'return' to terminate the current control flow, which is valid in CommonJS. When the example was copied and modified to use MJS syntax, the 'return' statement was left in but is not allowed. Refs: https://github.com/nodejs/node/pull/37594 PR-URL: https://github.com/nodejs/node/pull/39949 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
* src: add option to disable loading native addonsDominic Elm2021-09-2223-9/+272
| | | | | | | | PR-URL: https://github.com/nodejs/node/pull/39977 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Bradley Farias <bradley.meck@gmail.com> Reviewed-By: Guy Bedford <guybedford@gmail.com> Reviewed-By: Michael Dawson <midawson@redhat.com>
* crypto: fix RSA-PSS default saltLengthTobias Nießen2021-09-212-2/+44
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/39999 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com>
* crypto: fix default MGF1 hash for OpenSSL 3Tobias Nießen2021-09-212-2/+33
| | | | | | | | Refs: https://github.com/nodejs/node/pull/39999 PR-URL: https://github.com/nodejs/node/pull/40031 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com>
* meta: add more mailmap entries for bajtosRich Trott2021-09-212-3/+4
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/40023 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
* deps: patch for v8 on windowsJiawen Geng2021-09-214-6/+10
| | | | | | PR-URL: https://github.com/nodejs/node/pull/40010 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* tools: update remark-html to v13.0.2Michaël Zasso2021-09-213-9/+9
| | | | | | PR-URL: https://github.com/nodejs/node/pull/40043 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
* repl: fix top level await with surrogate charactersMestery2021-09-212-2/+19
| | | | | | | | | Fixes: https://github.com/nodejs/node/issues/39929 PR-URL: https://github.com/nodejs/node/pull/39931 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Shingo Inoue <leko.noor@gmail.com>
* lib: use standard property namesnull2021-09-211-14/+14
| | | | | | | | | | | | | The standard property names that aren't strings can be used where appropiate, this is one of them. PR-URL: https://github.com/nodejs/node/pull/39981 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Michael Dawson <midawson@redhat.com>
* doc: fix property name 'detail' of performanceEntryChristian Boehlke2021-09-211-6/+6
| | | | | | | | PR-URL: https://github.com/nodejs/node/pull/40019 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de>
* Revert "src: skip test_fatal/test_threads for Debug builds"Anna Henningsen2021-09-211-4/+0
| | | | | | | | | | | This reverts commit 1fc4d43a3255273709f5fcb51bb0c5b0407de4ac. PR-URL: https://github.com/nodejs/node/pull/39954 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* src: use Isolate::TryGetCurrent where appropriateAnna Henningsen2021-09-214-4/+4
| | | | | | | | | | | | | | In two places, we call `Isolate::GetCurrent()` even though that is technically invalid usage of the function. Now that V8 exposes `Isolate::TryGetCurrent()`, we can do this in a proper way. PR-URL: https://github.com/nodejs/node/pull/39954 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
* deps: update Acorn to v8.5.0Michaël Zasso2021-09-2112-526/+663
| | | | | | | | PR-URL: https://github.com/nodejs/node/pull/40015 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* src: make napi_create_reference accept symbolJckXia2021-09-213-3/+25
| | | | | | PR-URL: https://github.com/nodejs/node/pull/39926 Reviewed-By: Michael Dawson <midawson@redhat.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* doc: fix list indentation in corepack.mdAlexey Ten2021-09-211-9/+9
| | | | | | | | | PR-URL: https://github.com/nodejs/node/pull/40029 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Shingo Inoue <leko.noor@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
* crypto: check webcrypto asymmetric key types during importKeyFilip Skokan2021-09-216-3/+150
| | | | | PR-URL: https://github.com/nodejs/node/pull/39962 Reviewed-By: James M Snell <jasnell@gmail.com>
* build: add YAML linting to GitHub ActionsRich Trott2021-09-211-1/+17
| | | | | | | | PR-URL: https://github.com/nodejs/node/pull/40007 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com>
* build: add YAML lintingRich Trott2021-09-213-5/+33
| | | | | | | | PR-URL: https://github.com/nodejs/node/pull/40007 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com>
* tools,build: update YAML files in preparation for lintingRich Trott2021-09-215-16/+16
| | | | | | | | | | | Fix indentation, traiiling spaces, and missing newline issues in preparation for linting. PR-URL: https://github.com/nodejs/node/pull/40007 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com>
* build: run AUTHORS update weeklyRich Trott2021-09-211-0/+4
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/40004 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
* test: make tests pass on Windows with Unix EOLMichaël Zasso2021-09-214-8/+14
| | | | | | | | | | | | It is possible on Windows to configure Git to checkout line-endings "as-is", which for Node.js means Unix-style. This change makes the tests that rely on line endings pass in that case. PR-URL: https://github.com/nodejs/node/pull/40002 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
* src: fix C4805 MSVC warningMichaël Zasso2021-09-211-1/+1
| | | | | | | | | '==' : unsafe mix of type 'bool' and type 'int' in operation PR-URL: https://github.com/nodejs/node/pull/39998 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de>
* lib,repl: ignore non-canBeRequiredByUsers built-inXadillaX2021-09-214-5/+44
| | | | | | | | | e.g. `wasi` under no `--experimental-wasi-unstable-preview1` flag shouldn't be pre-required. PR-URL: https://github.com/nodejs/node/pull/39942 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
* meta: consolidate AUTHORS entries for mithunsasidharanRich Trott2021-09-212-1/+1
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/40003 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
* doc: fix missing history version in `fs.md`Antoine du Hamel2021-09-211-4/+12
| | | | | | | | | | | | | | Refs: https://github.com/nodejs/node/pull/33716 Refs: https://github.com/nodejs/node/pull/35993 Refs: https://github.com/nodejs/node/pull/35911 PR-URL: https://github.com/nodejs/node/pull/39972 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de>
* src: register external references of PipeWrap for snapshotJoyee Cheung2021-09-213-1/+18
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/39961 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Michael Dawson <midawson@redhat.com>
* src: register external references of TTYWrap for snapshotJoyee Cheung2021-09-213-0/+13
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/39961 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Michael Dawson <midawson@redhat.com>
* src: register external references of TCPWrap for snapshotJoyee Cheung2021-09-213-1/+24
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/39961 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Michael Dawson <midawson@redhat.com>
* src: register external references of SignalWrap for snapshotJoyee Cheung2021-09-212-0/+10
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/39961 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Michael Dawson <midawson@redhat.com>
* src: register missing process methods external referencesJoyee Cheung2021-09-211-0/+5
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/39961 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Michael Dawson <midawson@redhat.com>
* src: register missing stream wrap external referencesJoyee Cheung2021-09-211-0/+4
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/39961 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Michael Dawson <midawson@redhat.com>
* src: register external references of BaseObject for snapshotJoyee Cheung2021-09-213-7/+14
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/39961 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Michael Dawson <midawson@redhat.com>
* src: register external references of node-report for snapshotJoyee Cheung2021-09-212-0/+22
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/39961 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Michael Dawson <midawson@redhat.com>
* src: register external references of dtrace for snapshotJoyee Cheung2021-09-212-10/+30
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/39961 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Michael Dawson <midawson@redhat.com>
* meta: update AUTHORSNode.js GitHub Bot2021-09-101-0/+5
| | | | | | | PR-URL: https://github.com/nodejs/node/pull/39957 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* crypto: add rsa-pss keygen parametersFilip Skokan2021-09-105-22/+184
| | | | | PR-URL: https://github.com/nodejs/node/pull/39927 Reviewed-By: James M Snell <jasnell@gmail.com>
* build: preserves symbols during LTO with macOS linkerJesse Chan2021-09-101-0/+13
| | | | | | | | | | | | | | | | | | | | | | man ld -export_dynamic: ``` Preserves all global symbols in main executables during LTO. Without this option, Link Time Optimization is allowed to inline and remove global functions. This option is used when a main executable may load a plug-in which requires certain symbols from the main executable. ``` Bug: vercel/pkg#1155 Signed-off-by: Jesse Chan <jc@linux.com> PR-URL: https://github.com/nodejs/node/pull/39839 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com>
* stream: add signal support to pipeline generatorsRobert Nagy2021-09-107-33/+117
| | | | | | | | | | Generators in pipeline must be able to be aborted or pipeline can deadlock. PR-URL: https://github.com/nodejs/node/pull/39067 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
* Working on v16.9.2Richard Lau2021-09-101-2/+2
| | | | PR-URL: https://github.com/nodejs/node/issues/40069
* 2021-09-10, Version 16.9.1 (Current)v16.9.1v16.9.1-proposalRichard Lau2021-09-103-2/+15
| | | | | | | | Notable changes: This release fixes a regression introduced by the V8 9.3 update in Node.js 16.9.0. PR-URL: https://github.com/nodejs/node/issues/40069
* deps: V8: cherry-pick 9a607043cb31Jiawen Geng2021-09-102-3/+3
| | | | | | | | | | | | | | | | | | | | | | | Original commit message: [compiler] Gracefully handle an unsupported situation ... by skipping the optimization instead of CHECK-failing. Bug: v8:12188 Change-Id: I6709bf1c55506f3d12886efbfbb9934788cd02ce Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3148132 Auto-Submit: Georg Neis <neis@chromium.org> Commit-Queue: Jakob Gruber <jgruber@chromium.org> Reviewed-by: Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/main@{#76741} Refs: https://github.com/v8/v8/commit/9a607043cb3161f8ceae1583807bece595388108 PR-URL: https://github.com/nodejs/node/pull/40046 Fixes: https://github.com/nodejs/node/issues/40030 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Richard Lau <rlau@redhat.com>
* Working on v16.9.1Michaël Zasso2021-09-071-2/+2
| | | | PR-URL: https://github.com/nodejs/node/pull/40011
* 2021-09-07, Version 16.9.0 (Current)v16.9.0v16.9.0-proposalMichaël Zasso2021-09-068-9/+124
| | | | | | | | | | | | | | | | Notable changes: crypto: * (SEMVER-MINOR) add RSA-PSS params to asymmetricKeyDetails (Tobias Nießen) https://github.com/nodejs/node/pull/39851 deps: * (SEMVER-MINOR) add corepack (Maël Nison) https://github.com/nodejs/node/pull/39608 * (SEMVER-MINOR) update V8 to 9.3.345.16 (Michaël Zasso) https://github.com/nodejs/node/pull/39947 module: * (SEMVER-MINOR) support pattern trailers (Guy Bedford) https://github.com/nodejs/node/pull/39635 stream: * (SEMVER-MINOR) add stream.compose (Robert Nagy) https://github.com/nodejs/node/pull/39029 PR-URL: https://github.com/nodejs/node/pull/40011