V17.8.1 proposal #42612
Closed
V17.8.1 proposal #42612
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
resolve: #40330 PR-URL: #42377 Fixes: #40330 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Mestery <[email protected]>
Property name `min` was updated in 23637e9 to `lowest` but an instance of `min` was missed. PR-URL: #42398 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Harshitha K P <[email protected]>
Refs: #39043 PR-URL: #42394 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Akhil Marsonya <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
Improve the function's structure and clarify the special handling of ChaCha20-Poly1305. Remove the IS_OCB_MODE macro. PR-URL: #42368 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
PR-URL: #42323 Reviewed-By: James M Snell <[email protected]>
PR-URL: #42396 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
PR-URL: #42401 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Akhil Marsonya <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
Promises are never destroyed manually therefore it's not needed to attach an object to track if destroy hook was called already. PR-URL: #42402 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
PR-URL: #42165 Fixes: #42286 Reviewed-By: Antoine du Hamel <[email protected]>
Signed-off-by: Michael Dawson <[email protected]> PR-URL: #42284 Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: James M Snell <[email protected]>
PR-URL: #42416 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
Cherry-pick 12c8b4d Original commit message: This commit is a suggestion for adding a rule for NULL usages in the code base. This will currently report a number of errors which could be ignored using // NOLINT (readability/null_usage) PR-URL: #17373 Reviewed-By: Jon Moss <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Timothy Gu <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Refs: 12c8b4d Cherry-pick fc81e80 Original commit message: Update cpplint.py to check for inline headers when the corresponding header is already included. PR-URL: #21521 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: James M Snell <[email protected]> Refs: fc81e80 Cherry-pick cbc3dd9 Original commit message: src, tools: add check for left leaning pointers This commit adds a rule to cpplint to check that pointers in the code base lean to the left and not right, and also fixes the violations reported. PR-URL: #21010 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: James M Snell <[email protected]> Refs: cbc3dd9 Cherry-pick 9029981 Original commit message: tools: fix cpplint.py header rules THIS COMMIT SHOULD GO WITH THE NEXT. IT WILL FIND NEW LINT. PR-URL: #26306 Reviewed-By: Gireesh Punathil <[email protected]> Refs: 9029981 Cherry-pick 0a25ace Original commit message: tools: move cpplint configuration to .cpplint PR-URL: #27098 Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Daniel Bevenius <[email protected]> Refs: 0a25ace Cherry-pick afa9a72 Original commit message: tools: refloat update link to google styleguide for cpplint This commit updates two old links to Google's C++ styleguide which currently result in a 404 when accessed. PR-URL: #30876 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: David Carlier <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Rich Trott <[email protected]> Refs: afa9a72 Cherry-pick e23bf8f Original commit message: tools,src: refloat forbid usage of v8::Persistent `v8::Persistent` comes with the surprising catch that it requires manual cleanup. `v8::Global` doesn’t, making it easier to use, and additionally provides move semantics. New code should always use `v8::Global`. PR-URL: #31018 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: David Carlier <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Stephen Belanger <[email protected]> PR-URL: #35569 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Daijiro Wachi <[email protected]> Reviewed-By: Jiawen Geng <[email protected]> PR-URL: #35719 Reviewed-By: Antoine du Hamel <[email protected]> PR-URL: #35866 PR-URL: #36213 Reviewed-By: Franziska Hinkelmann <[email protected]> PR-URL: #36235 Reviewed-By: Luigi Pinca <[email protected]> PR-URL: #36324 Reviewed-By: Beth Griggs <[email protected]> PR-URL: #38851 Reviewed-By: Khaidi Chu <[email protected]> PR-URL: #42416 Reviewed-By: Mestery <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
Instead of passing the snapshot blob, the per-isolate data indices and the EnvSerializeInfo separately, use the aggregate type Snapshot to carry these around, and refactor NodeMainInstance so that it owns the v8::Isolate::CreateParams when it owns its isolate. This also gets rid of the owns_isolate_ and deserialize_mode_ booleans in NodeMainInstance since NodeMainInstance can compute these by just checking if it has pointers to the CreateParams or the SnapshotData. PR-URL: #42360 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
PR-URL: #42346 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]>
PR-URL: #42444 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Tierney Cyren <[email protected]> Reviewed-By: Mestery <[email protected]>
PR-URL: #42414 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Rich Trott <[email protected]>
PR-URL: #42428 Reviewed-By: Mestery <[email protected]> Reviewed-By: Harshitha K P <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Beth Griggs <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Akhil Marsonya <[email protected]> Reviewed-By: Tierney Cyren <[email protected]>
PR-URL: #42455 Reviewed-By: Mestery <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
Signed-off-by: Darshan Sen <[email protected]> PR-URL: #42054 Reviewed-By: Matteo Collina <[email protected]>
Fixes: #36162 PR-URL: #42339 Reviewed-By: Jacob Smith <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]> Reviewed-By: James M Snell <[email protected]>
PR-URL: #42427 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Filip Skokan <[email protected]>
PR-URL: #42488 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
We currently defer finalizer callbacks until the loop is idle. Warn users that the weak reference returned by napi_wrap() isn't guaranteed to be valid just because the finalizer hasn't yet been called. PR-URL: #42461 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Tierney Cyren <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: James M Snell <[email protected]>
PR-URL: #42487 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
Fix issue that will be flagged when we update ESLint and dependencies. PR-URL: #42489 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
PR-URL: #42489 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
PR-URL: #42484 Reviewed-By: Mestery <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
Git for Windows creates read-only files inside the .git directory. fs.rm() was built in a way, to work around any EPERM error that can happen while deleting the .git directory by turning the directory into a writable one. This change adds a regression test for that. Refs: isaacs/rimraf#21 Refs: #38810 (comment) Signed-off-by: Darshan Sen <[email protected]> PR-URL: #42410 Reviewed-By: Ben Coe <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
https://github.com/actions/checkout\#checkout-v3 PR-URL: #42460 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Jiawen Geng <[email protected]> Reviewed-By: Tierney Cyren <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
PR-URL: #42465 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
PR-URL: #42500 Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
The `var` keyword is known to be problematic and is not needed here, so better to use the `let` keyword for variable declarations. PR-URL: #42563 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Akhil Marsonya <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]>
Fixes: #42417 PR-URL: #42522 Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: James M Snell <[email protected]>
PR-URL: #42588 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Mestery <[email protected]>
This sentence implies that AES-128 is preferred over AES-256 because of a related-key attack from 2009. However, that attack by Alex Biryukov, Orr Dunkelman, Nathan Keller, Dmitry Khovratovich, and Adi Shamir, while impressive, is only effective against variants of AES-256 with a reduced number of rounds and it requires related keys. This means that the attack is not effective against AES-256 as it is used within TLS. (AES-128 is still often preferred over AES-256 simply because it is believed to be sufficiently secure and because it is faster.) PR-URL: #42578 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
Linting markdown runs the JavaScript linting job, so consolidate them to conserve time and resources. Run JavaScript separately, but then run markdown linting so it can use the cached results of the JavaScript linting. PR-URL: #42572 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
PR-URL: #42573 Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Akhil Marsonya <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
PR-URL: #42573 Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Akhil Marsonya <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
The FS docs wrongfully indicated support for passing object with an own `toString` function property to `FileHandle.prototype.appendFile`, `FileHandle.prototype.writeFile`, `FileHandle.prototype.write`, `fsPromises.writeFile`, and `fs.writeSync`. This commit fixes that, and adds some test to ensure the actual behavior is aligned with the docs. It also fixes a bug that makes the process crash if a non-buffer object was passed to `FileHandle.prototype.write`. Refs: #34993 PR-URL: #41677 Refs: #41666 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: James M Snell <[email protected]>
resolve: #38856 PR-URL: #42338 Fixes: #38856 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: James M Snell <[email protected]>
Refs: #42599 (comment) PR-URL: #42602 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Xuguang Mei <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
|
Review requested:
|
nodejs-github-bot
added
build
juanarbol
force-pushed
the
v17.8.1-proposal
branch
from
fde30da to
f5f29f4
Compare
richardlau
reviewed
Apr 5, 2022
| @@ -44,6 +45,108 @@ | |||
| * [io.js](CHANGELOG_IOJS.md) | |||
| * [Archive](CHANGELOG_ARCHIVE.md) | |||
|
|
|||
| <a id="17.8.1"></a> | |||
|
|
|||
| ## 2022-04-04, Version 17.8.1 (Current), @BethGriggs prepared by @juanarbol | |||
There was a problem hiding this comment.
Since this includes a semver-minor the version should be 17.9.0.
|
Moved to: #42613 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
2022-04-04, Version 17.8.1 (Current), @BethGriggs prepared by @juanarbol
Notable Changes
f3c3479bf0] - doc: add @meixg to collaborators (Xuguang Mei) #42576Commits
e73bf4355d] - async_hooks: remove destroyed symbol on Promises (Gerhard Stöbich) #4240294e8da8f9e] - bootstrap: reset process._exit and process.exitCode in pre-execution (Joyee Cheung) #42466cbdc0ab9c7] - bootstrap: run inspector and event loop in snapshot builder (Joyee Cheung) #42466fa28439583] - bootstrap: make I/O streams work with user-land snapshot (Joyee Cheung) #42466a113d4379a] - bootstrap: use SnapshotData to pass snapshot data around (Joyee Cheung) #423603fbfd6550d] - buffer: improve Blob constructor error message when passing a string (Xuguang Mei) #42338b250b8ec95] - buffer: fixatobinput validation (Antoine du Hamel) #42539ebefd25141] - build: consolidate JS and md linting GitHub Actions (Rich Trott) #42572fe62d4b506] - build: set stale action back to running nightly (Michael Dawson) #425496b14bd8f31] - build: add --node-snapshot-main configure option (Joyee Cheung) #42466b647336c96] - build: bump actions/checkout (Eliaz Bobadilla) #424607105f8b467] - child_process: add env contents types in JSDoc (Rich Trott) #42494fa7edb7acd] - crypto: do not add undefined hash in webcrypto normalizeAlgorithm (Filip Skokan) #42559c036258809] - crypto: cleanup webcrypto jwk code (Filip Skokan) #425624e7b7465bc] - crypto: fix webcrypto derive key lengths (Filip Skokan) #425422ca08c948c] - (SEMVER-MINOR) crypto: make authTagLength optional for CC20P1305 (Tobias Nießen) #424279158b91440] - deps: update undici to 4.16.0 (Node.js GitHub Bot) #42414e158a8db98] - doc: remove faulty justification for 128-bit AES (Tobias Nießen) #4257873a1d224c0] - doc: fix documentation ofFileHandle.prototype.appendFile(Antoine du Hamel) #42588858af420ef] - doc: change "OCSP Request" to "OCSP request" (Tobias Nießen) #42582a6c65fc7d2] - doc: aes webcrypto unwrap is not a node-specific extensions (Filip Skokan) #42561d2f17ace5d] - doc: simplify recommendations in process.md (Rich Trott) #425568dd88cc269] - doc: clarify recommendations in stream.md (Rich Trott) #42555b7748e4d89] - doc: simplify recommendation in webcrypto.md (Rich Trott) #425548d5c9d9c24] - doc: update DEP0102 text (Rich Trott) #425538e55e59cb4] - doc: remove util.promisify() content in readline.md (Rich Trott) #425520c1e6905d5] - doc: add introduction sentence for CJS (Antoine du Hamel) #42491f3c3479bf0] - doc: add @meixg to collaborators (Xuguang Mei) #4257653d73047a9] - doc: consolidate CI sections (Rich Trott) #42534ae9d5fdb39] - doc: document breaking change inhttp.IncomingMessage'close'event (Paolo Insogna) #4252198f1f34cbe] - doc: remove extraneous comma (Rich Trott) #425488b7c057ff1] - doc: guide towards x509.fingerprint256 (Tobias Nießen) #42516d2ebe2b190] - doc: fix internal link in collaborator-guide.md (Daeyeon Jeong) #42551607a5ca653] - doc: add suggestion for OpenSSL only sec releases (Michael Dawson) #42456838b143f69] - doc: fix comment text in async_hooks example (Rich Trott) #42499743b62b4d5] - doc: addstabilityclass to legacy status description (Daniel Roe) #425259e14b0efc2] - doc: suggest checkHost in checkServerIdentity docs (Tobias Nießen) #42495a946977de7] - doc: update security release onboarding (Joe Sepi) #42333efd30710be] - doc: fix question promise API example (Xuguang Mei) #4246528a770eec0] - doc: remove comma splice in events.md (Rich Trott) #42484f9f6c8e001] - doc: clarify napi_finalize behavior (Alba Mendez) #4246154615fdce1] - doc: expand history for conditional exports changes in v12 (Greg Poole) #423393b810caddc] - doc: change comma-splice to two sentences (Rich Trott) #4245513840ef2af] - doc: add link to section (Rich Trott) #4242826d5e580b5] - doc: fix typo in async_context.md (Anupama Codippily) #4244441b353d0c8] - doc: addtrace\_gcto diagnostic tooling support document (Tony Gorez) #4234689da27c11f] - doc: make header smaller and dropdown click-driven when JS is on (Paolo Insogna) #421659b0ac65971] - doc: standardize typography for semantic versioning (Rich Trott) #4240176be5651af] - doc: unify import order in CCM example (Tobias Nießen) #4239407b38f6695] - doc: update property name (Rich Trott) #423985602c08a12] - doc,test: clarify ChaCha20-Poly1305 usage (Tobias Nießen) #42323eea25dc430] - esm: emit experimental warnings in common place (Jacob Smith) #4231499f9880a6f] - fs: fix write methods param validation and docs (Livia Medeiros) #416775e0ca4c14b] - lib: prepare files for no-var lint rule (Rich Trott) #425736258055603] - lib: source maps filter null prefix (Fabian Cook) #425223ee5434141] - lib: improve the coverage of the validator (mawaregetsuka) #4244386a05ca604] - lib: update JSDoc for linting (Rich Trott) #424891e3e6895a6] - meta: update .mailmap and AUTHORS (Rich Trott) #42602dea7f22bdb] - meta: move one or more collaborators to emeritus (Node.js GitHub Bot) #42500466673735d] - meta: update AUTHORS (Node.js GitHub Bot) #42585439a87ec36] - meta: update AUTHORS (Node.js GitHub Bot) #424887956559fe4] - net,dns: trace tcp connection and dns by perf_hooks (theanarkh) #4239093c874fdbe] - node-api: format Node-API related code (Vladimir Morozov) #423960fcb067991] - os: avoid unnecessary usage of var (Mohammed Keyvanzadeh) #425633084af316c] - src: add proper mutexes for accessing FIPS state (Anna Henningsen) #422785c9771f8e9] - src: fix typo in InspectorIoDelegate constructor (Kohei Ueno) #42520987b5d13cd] - src: remove unnecessary static qualifier in crypto_dh.cc (Darshan Sen) #42492c9c0cc5b9e] - src: address 3 useless call coverity warnings (Michael Dawson) #424268e8f129a5e] - src: properly report exceptions from AddressToJS() (Darshan Sen) #42054c516c84e52] - src: suppress false coverity warning (Michael Dawson) #422840a2123d24c] - src: refactor IsSupportedAuthenticatedMode (Tobias Nießen) #42368899de04a26] - src,crypto: handle empty maybe correctly in crypto_dh.cc (Darshan Sen) #42492cf1543e71e] - src,crypto: remove uses of AllocatedBuffer from crypto_dh.cc (Darshan Sen) #4249244ac5e8ab3] - test: improve lib/internal/readline/promises.js coverage (MURAKAMI Masahiko) #42420dd1ea775ac] - test: remove hack foratobandbtoaWPT tests (Antoine du Hamel) #42540d71df7a156] - test: pass data into napi_create_external (Joyee Cheung) #425323734b64f77] - test: improveFileHandle.prototype.writecoverage (Antoine du Hamel) #42541f25ba8c4c7] - test: add test for exception handlings in debugger (Kohei Ueno) #42327e310a2701d] - test: fix typo in common/wpt.js (Ikko Ashimine) #4256737ad634bae] - test: fix comments in test files (Daeyeon Jeong) #425368bb04f5b0b] - test: fix typos in test/parallel (Daeyeon Jeong) #42502b756ed8649] - test: add trace-gc flag test (Tony Gorez) #42471470e911d1c] - test,fs: add fs.rm() tests for .git directories (Darshan Sen) #424100a8b362729] - tools: enable no-var ESLint rule for lib (Rich Trott) #425734dfc29dbda] - tools: fixed bug causing JSON format to be broken (mawaregetsuka) #4156516702660a4] - tools: update GHA actions version (Antoine du Hamel) #42498f4decbd608] - tools: update eslint to 8.12.0 (Node.js GitHub Bot) #42489136b9f8cdc] - tools: update lint-md-dependencies to [email protected] (Node.js GitHub Bot) #42487bd688f7513] - tools: refloat 7 Node.js patches to cpplint.py (Rich Trott) #4241680a22516a6] - tools: bump cpplint to 1.6.0 (Rich Trott) #4241617770bea06] - tools: fix skip PR if CI is still running (Xuguang Mei) #42377