- Aug 03, 2021
-
-
Michaël Zasso authored
Notable changes: * Updated npm to 7.20.3 (npm team) https://github.com/nodejs/node/pull/39579 * Reverted an ABI-breaking change from V8 9.2 that could impact some native modules (Michaël Zasso) https://github.com/nodejs/node/pull/39624 * Fixed a bug in error handling known to affect at least Webpack and Jest (Guy Bedford) https://github.com/nodejs/node/pull/39593 PR-URL: https://github.com/nodejs/node/pull/39631
-
Rich Trott authored
Apply 8ec18cf: "Support STRING16 in the template when converting CBOR map keys" Refs: https://chromium.googlesource.com/deps/inspector_protocol/+/8ec18cf0885bef0b5c2a922c5dc3813cbf63e962 We're over 2 years out of date in the tools/inspector_protocol directory and I have to imagine this will come back to bite us at some point. But I also don't want to do a huge update all at once, so starting with a single commit. I might bundle commits together a bit more if this goes well. PR-URL: https://github.com/nodejs/node/pull/39614 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Tobias Nießen <tniessen@tnie.de>
-
Rich Trott authored
Remove unnecessary name specificity in mailmap entries. AUTHORS is unchanged after running update-authors.js with these changes. PR-URL: https://github.com/nodejs/node/pull/39612 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com> Reviewed-By:
Anna Henningsen <anna@addaleax.net>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/39611 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/39611 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/39611 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/39629 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Tobias Nießen <tniessen@tnie.de> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com>
-
Luigi Pinca authored
PR-URL: https://github.com/nodejs/node/pull/39602 Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Shingo Inoue <leko.noor@gmail.com>
-
Luigi Pinca authored
This is basically a revert of https://github.com/nodejs/node/commit/c452632d3421d6baf26a61e14c21c922bd18175e. The problem with that commit is that the promises returned by `once(server, 'stream')` are all resolved with the same stream when the first `'stream'` event is emitted. Refs: https://github.com/nodejs/node/pull/39525#issuecomment-889080913 PR-URL: https://github.com/nodejs/node/pull/39591 Reviewed-By:
Rich Trott <rtrott@gmail.com> Reviewed-By:
Anna Henningsen <anna@addaleax.net>
-
- Aug 02, 2021
-
-
Rich Trott authored
This consolidates two email addresses/entries in AUTHORS into one. PR-URL: https://github.com/nodejs/node/pull/39596 Reviewed-By:
Ujjwal Sharma <ryzokuken@disroot.org> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com>
-
Rich Trott authored
This consolidates two email addresses/entries in AUTHORS into one. PR-URL: https://github.com/nodejs/node/pull/39596 Reviewed-By:
Ujjwal Sharma <ryzokuken@disroot.org> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com>
-
Rich Trott authored
This consolidates two email addresses/entries in AUTHORS into one. PR-URL: https://github.com/nodejs/node/pull/39596 Reviewed-By:
Ujjwal Sharma <ryzokuken@disroot.org> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com>
-
Himadri Ganguly authored
PR-URL: https://github.com/nodejs/node/pull/39604 Reviewed-By:
Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By:
Tobias Nießen <tniessen@tnie.de> Reviewed-By:
Zijian Liu <lxxyxzj@gmail.com>
-
Robert Nagy authored
Adds did read accessor used to determine whether a readable has been read from. PR-URL: https://github.com/nodejs/node/pull/39589 Refs: https://github.com/nodejs/undici/pull/907 Reviewed-By:
Matteo Collina <matteo.collina@gmail.com> Reviewed-By:
Benjamin Gruenbaum <benjamingr@gmail.com>
-
Antoine du Hamel authored
PR-URL: https://github.com/nodejs/node/pull/39504 Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Michaël Zasso <targos@protonmail.com>
-
James M Snell authored
Signed-off-by:
James M Snell <jasnell@gmail.com> Fixes: https://github.com/nodejs/node/issues/39548 PR-URL: https://github.com/nodejs/node/pull/39550 Reviewed-By:
Bryan English <bryan@bryanenglish.com> Reviewed-By:
Chengzhong Wu <legendecas@gmail.com>
-
- Aug 01, 2021
-
-
Anna Henningsen authored
For every object that inherits from `Object.prototype`, the REPL includes the `Object.prototype` methods in its autocompletion. This is already a little noisy, but in particular, this also includes the legacy `__defineGetter__` family of methods; since those are deprecated and not in practical use anymore, it helps reduce noise a bit to remove them. This commit does not remove `__proto__` as it is a little more popular and, despite its downsides, a slightly more convenient way to access the prototype of an object in the REPL than `Object.getPrototypeOf(...)`. PR-URL: https://github.com/nodejs/node/pull/39576 Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Gus Caplan <me@gus.host> Reviewed-By:
Tobias Nießen <tniessen@tnie.de> Reviewed-By:
Colin Ihrig <cjihrig@gmail.com>
-
Rich Trott authored
This adds test coverage to `unpackError()` in `lib/internal/debugger/inspect_client.js`. That function previously was untested. PR-URL: https://github.com/nodejs/node/pull/39570 Refs: https://github.com/nodejs/node-inspect/issues/101 Reviewed-By:
Jan Krems <jan.krems@gmail.com> Reviewed-By:
Michaël Zasso <targos@protonmail.com>
-
Rich Trott authored
The data parameter of unpackError() is typically undefined. PR-URL: https://github.com/nodejs/node/pull/39570 Refs: https://github.com/nodejs/node-inspect/issues/101 Reviewed-By:
Jan Krems <jan.krems@gmail.com> Reviewed-By:
Michaël Zasso <targos@protonmail.com>
-
Mestery authored
Use ERR_ILLEGAL_CONSTRUCTOR error instead of `illegal constructor` or `Illegal constructor` TypeError. PR-URL: https://github.com/nodejs/node/pull/39556 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com> Reviewed-By:
Darshan Sen <raisinten@gmail.com>
-
Rich Trott authored
Refs: https://github.com/nodejs/node/issues/39555 PR-URL: https://github.com/nodejs/node/pull/39557 Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Michaël Zasso <targos@protonmail.com>
-
Ash Cripps authored
Update the minimum macos version that can compile to match the xcode requirements. Also move mac arm64 to tier 1. refs: https://github.com/nodejs/node/issues/39584#issuecomment-889701855 PR-URL: https://github.com/nodejs/node/pull/39586 Reviewed-By:
Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By:
Beth Griggs <bgriggs@redhat.com> Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Richard Lau <rlau@redhat.com> Reviewed-By:
Jiawen Geng <technicalcute@gmail.com> Reviewed-By:
Michaël Zasso <targos@protonmail.com>
-
Richard Lau authored
Fixes doc build warnings: ``` Failed to add alternative version links to webstreams Failed to add alternative version links to async_context ``` PR-URL: https://github.com/nodejs/node/pull/39575 Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Rich Trott <rtrott@gmail.com> Reviewed-By:
Tobias Nießen <tniessen@tnie.de> Reviewed-By:
Beth Griggs <bgriggs@redhat.com> Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com> Reviewed-By:
Michael Dawson <midawson@redhat.com> Reviewed-By:
Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By:
Darshan Sen <raisinten@gmail.com>
-
Rich Trott authored
The email address for Weijia Wang in the AUTHORS and .mailmap files is different from the email address in the README.md file. Based on their GitHub profile and commit metadata, the one in the README.md file is the preferred email. Updating .mailmap and AUTHORS to reflect it. PR-URL: https://github.com/nodejs/node/pull/39505 Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Weijia Wang <starkwang@126.com>
-
- Jul 31, 2021
-
-
Rich Trott authored
Ignore all markdown files when determining if test-asan, coverage-*, and test-macos need to run. Refs: https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#patterns-to-match-file-paths PR-URL: https://github.com/nodejs/node/pull/39565 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Richard Lau <rlau@redhat.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:
Tobias Nießen <tniessen@tnie.de>
-
Guy Bedford authored
PR-URL: https://github.com/nodejs/node/pull/39593 Reviewed-By:
Bradley Farias <bradley.meck@gmail.com> Reviewed-By:
Rich Trott <rtrott@gmail.com>
-
Rich Trott authored
Add mailmap entry for garygsc and re-run AUTHORS to remove duplicate entry. PR-URL: https://github.com/nodejs/node/pull/39588 Reviewed-By:
Beth Griggs <bgriggs@redhat.com> Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com>
-
Rich Trott authored
Add mailmap entry for ttzztztz and re-run AUTHORS to remove duplicate entry. PR-URL: https://github.com/nodejs/node/pull/39588 Reviewed-By:
Beth Griggs <bgriggs@redhat.com> Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/39587 Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Tobias Nießen <tniessen@tnie.de>
-
- Jul 30, 2021
-
-
Juan José Arboleda authored
PR-URL: https://github.com/nodejs/node/pull/39491 Reviewed-By:
James M Snell <jasnell@gmail.com>
-
legendecas authored
PR-URL: https://github.com/nodejs/node/pull/39532 Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Khaidi Chu <i@2333.moe>
-
npm team authored
PR-URL: https://github.com/nodejs/node/pull/39579 Reviewed-By:
Myles Borins <myles.borins@gmail.com> Reviewed-By:
Beth Griggs <bgriggs@redhat.com>
-
Jordan Harband authored
PR-URL: https://github.com/nodejs/node/pull/39583 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Beth Griggs <bgriggs@redhat.com> Reviewed-By:
Ash Cripps <acripps@redhat.com>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/39561 Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Zijian Liu <lxxyxzj@gmail.com>
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/39560 Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Zijian Liu <lxxyxzj@gmail.com>
-
Beth Griggs authored
This is a security release. Notable Changes: - CVE-2021-22930: Use after free on close http2 on stream canceling (High) [#39423](https://github.com/nodejs/node/pull/39423) - (SEMVER-MINOR) deps: update V8 to 9.2.230.21 (Michaël Zasso) [#39470](https://github.com/nodejs/node/pull/39470) - inspector: mark as stable (Gireesh Punathil) [#37748](https://github.com/nodejs/node/pull/37748) - punycode: add pending deprecation (Antoine du Hamel) [#38444](https://github.com/nodejs/node/pull/38444) - (SEMVER-MINOR) repl: enable --experimental-repl-await /w opt-out (hemanth.hm) [#34733](https://github.com/nodejs/node/pull/34733) PR-URL: https://github.com/nodejs/node/pull/39534
-
Richard Lau authored
Move the section on `NODE_MODULE_VERSION` into the section for major releases as it should only be updated for a major release. Add a note to remove the `-pre` suffix from the registry for the release commit. PR-URL: https://github.com/nodejs/node/pull/39544 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Beth Griggs <bgriggs@redhat.com> Reviewed-By:
Danielle Adams <adamzdanielle@gmail.com> Reviewed-By:
James M Snell <jasnell@gmail.com>
-
Richard Lau authored
We no longer produced ARMv6 builds, and the ARMv7 builds are now cross compiled and are as quick as builds on the other platforms. PR-URL: https://github.com/nodejs/node/pull/39544 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Beth Griggs <bgriggs@redhat.com> Reviewed-By:
Danielle Adams <adamzdanielle@gmail.com> Reviewed-By:
James M Snell <jasnell@gmail.com>
-
Richard Lau authored
Put the command examples for finding and replacing the `REPLACEME` tags into code fences so that they are more easily copiable when viewing the guide in the GitHub web UI. PR-URL: https://github.com/nodejs/node/pull/39544 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Beth Griggs <bgriggs@redhat.com> Reviewed-By:
Danielle Adams <adamzdanielle@gmail.com> Reviewed-By:
James M Snell <jasnell@gmail.com>
-
Richard Lau authored
Add `backport-open-v1.x,backported-to-v1.x` labels to `branch-diff` commands as these indicate pull requests that are being manually backported and should not be cherry-picked. PR-URL: https://github.com/nodejs/node/pull/39544 Reviewed-By:
Michaël Zasso <targos@protonmail.com> Reviewed-By:
Beth Griggs <bgriggs@redhat.com> Reviewed-By:
Danielle Adams <adamzdanielle@gmail.com> Reviewed-By:
James M Snell <jasnell@gmail.com>
-