FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

v17.9.0 proposal by juanarbol · Pull Request #42613 · nodejs/node · GitHub

/ node Public

v17.9.0 proposal - #42613

Merged
BethGriggs merged 96 commits into
v17.xfrom
v17.9.0-proposal
Apr 7, 2022
ConversationCommits (96)ChecksFiles changed
Merged

v17.9.0 proposal#42613
BethGriggs merged 96 commits into
v17.xfrom
v17.9.0-proposal

Conversation

juanarbol commented Apr 5, 2022 •
edited
Loading

Copy link
Copy Markdown
Member

2022-04-07, Version 17.9.0 (Current), @BethGriggs prepared by @juanarbol

Notable Changes

Commits

  • [bb71433334] - async_hooks: remove destroyed symbol on Promises (Gerhard Stöbich) #42402
  • [b48a6cb3f9] - bootstrap: reset process._exit and process.exitCode in pre-execution (Joyee Cheung) #42466
  • [b89f038537] - bootstrap: run inspector and event loop in snapshot builder (Joyee Cheung) #42466
  • [177558600e] - bootstrap: make I/O streams work with user-land snapshot (Joyee Cheung) #42466
  • [e3683cb34d] - bootstrap: refresh options in pre-execution (Joyee Cheung) #42466
  • [d302d2f0d2] - bootstrap: use SnapshotData to pass snapshot data around (Joyee Cheung) #42360
  • [eb3dfc00f0] - buffer: improve Blob constructor error message when passing a string (Xuguang Mei) #42338
  • [f45d5537c1] - buffer: fix atob input validation (Antoine du Hamel) #42539
  • [fb6a5ba8d7] - build: remove precompiled header and debug information for host builds (Niyas Sait) #42538
  • [1f7d2e800c] - build: windows/arm64 native compilation support (Niyas Sait) #42408
  • [d9a1d7866c] - build: consolidate JS and md linting GitHub Actions (Rich Trott) #42572
  • [ecb5be845d] - build: set stale action back to running nightly (Michael Dawson) #42549
  • [f9fb7f6d96] - build: add --node-snapshot-main configure option (Joyee Cheung) #42466
  • [c6808f088b] - build: bump actions/checkout (Eliaz Bobadilla) #42460
  • [9a54acb7c6] - child_process: add env contents types in JSDoc (Rich Trott) #42494
  • [a2f07380ea] - crypto: do not add undefined hash in webcrypto normalizeAlgorithm (Filip Skokan) #42559
  • [9b4bd7d031] - crypto: cleanup webcrypto jwk code (Filip Skokan) #42562
  • [541a1328b0] - crypto: fix webcrypto derive key lengths (Filip Skokan) #42542
  • [7124f91cbf] - (SEMVER-MINOR) crypto: make authTagLength optional for CC20P1305 (Tobias Nießen) #42427
  • [30dc6dd3fb] - deps: update undici to 4.16.0 (Node.js GitHub Bot) #42414
  • [6e56924274] - doc: simplify Http2Stream encoding text (Rich Trott) #42597
  • [261672b1da] - doc: remove obsolete stream API selection text (Rich Trott) #42586
  • [beffed1880] - doc: remove faulty justification for 128-bit AES (Tobias Nießen) #42578
  • [71f4a39086] - doc: fix documentation of FileHandle.prototype.appendFile (Antoine du Hamel) #42588
  • [c83ea22f7c] - doc: change "OCSP Request" to "OCSP request" (Tobias Nießen) #42582
  • [71ab0dea35] - doc: aes webcrypto unwrap is not a node-specific extensions (Filip Skokan) #42561
  • [1c614184da] - doc: simplify recommendations in process.md (Rich Trott) #42556
  • [c036800ddc] - doc: clarify recommendations in stream.md (Rich Trott) #42555
  • [dcf0abf8c7] - doc: simplify recommendation in webcrypto.md (Rich Trott) #42554
  • [8333fa063b] - doc: update DEP0102 text (Rich Trott) #42553
  • [8b08bff682] - doc: remove util.promisify() content in readline.md (Rich Trott) #42552
  • [94492424ba] - doc: add introduction sentence for CJS (Antoine du Hamel) #42491
  • [f0fc2744a5] - doc: add @meixg to collaborators (Xuguang Mei) #42576
  • [d935fef594] - doc: consolidate CI sections (Rich Trott) #42534
  • [fd45df314b] - doc: document breaking change in http.IncomingMessage 'close' event (Paolo Insogna) #42521
  • [53584fa750] - doc: remove extraneous comma (Rich Trott) #42548
  • [b819af6509] - doc: guide towards x509.fingerprint256 (Tobias Nießen) #42516
  • [f2355e41ed] - doc: fix internal link in collaborator-guide.md (Daeyeon Jeong) #42551
  • [ffc6776996] - doc: add suggestion for OpenSSL only sec releases (Michael Dawson) #42456
  • [1454c0297d] - doc: fix comment text in async_hooks example (Rich Trott) #42499
  • [b9ab9867f4] - doc: add stability class to legacy status description (Daniel Roe) #42525
  • [6c13988d53] - doc: suggest checkHost in checkServerIdentity docs (Tobias Nießen) #42495
  • [28665a9dd6] - doc: update security release onboarding (Joe Sepi) #42333
  • [d335addf0c] - doc: fix question promise API example (Xuguang Mei) #42465
  • [7cf9febcb4] - doc: remove comma splice in events.md (Rich Trott) #42484
  • [3c3684d9f1] - doc: clarify napi_finalize behavior (Alba Mendez) #42461
  • [334cc1936b] - doc: expand history for conditional exports changes in v12 (Greg Poole) #42339
  • [fb146f9eaf] - doc: change comma-splice to two sentences (Rich Trott) #42455
  • [ce4b823946] - doc: add link to section (Rich Trott) #42428
  • [5869275479] - doc: fix typo in async_context.md (Anupama Codippily) #42444
  • [48bd9fa2c7] - doc: add trace_gc to diagnostic tooling support document (Tony Gorez) #42346
  • [00f693b6b1] - doc: make header smaller and dropdown click-driven when JS is on (Paolo Insogna) #42165
  • [abbb23620a] - doc: standardize typography for _semantic versioning_ (Rich Trott) #42401
  • [e763e575c6] - doc: unify import order in CCM example (Tobias Nießen) #42394
  • [10d638a735] - doc: update property name (Rich Trott) #42398
  • [5589a448b7] - doc,test: clarify ChaCha20-Poly1305 usage (Tobias Nießen) #42323
  • [902776e674] - esm: emit experimental warnings in common place (Jacob Smith) #42314
  • [8009cb0a78] - fs: fix write methods param validation and docs (Livia Medeiros) #42631
  • [a9dc3a92d9] - lib: prepare files for no-var lint rule (Rich Trott) #42573
  • [3306fee824] - lib: source maps filter null prefix (Fabian Cook) #42522
  • [3bac969655] - lib: improve the coverage of the validator (mawaregetsuka) #42443
  • [b74de21cc3] - lib: update JSDoc for linting (Rich Trott) #42489
  • [7766bf954f] - meta: update .mailmap and AUTHORS (Rich Trott) #42602
  • [93ffc5535a] - meta: move one or more collaborators to emeritus (Node.js GitHub Bot) #42500
  • [256509056d] - meta: update AUTHORS (Node.js GitHub Bot) #42585
  • [41c2a32390] - meta: update AUTHORS (Node.js GitHub Bot) #42488
  • [b71a8107c0] - net,dns: trace tcp connection and dns by perf_hooks (theanarkh) #42390
  • [f9f3b6e45d] - node-api: format Node-API related code (Vladimir Morozov) #42396
  • [0bd9d9e24f] - os: avoid unnecessary usage of var (Mohammed Keyvanzadeh) #42563
  • [e798e26dfd] - src: add proper mutexes for accessing FIPS state (Anna Henningsen) #42278
  • [a1fe0d2222] - src: fix typo in InspectorIoDelegate constructor (Kohei Ueno) #42520
  • [0c54f3637b] - src: remove unnecessary static qualifier in crypto_dh.cc (Darshan Sen) #42492
  • [2e6a66d7d7] - src: address 3 useless call coverity warnings (Michael Dawson) #42426
  • [ce9d840079] - src: properly report exceptions from AddressToJS() (Darshan Sen) #42054
  • [c6a558c61b] - src: suppress false coverity warning (Michael Dawson) #42284
  • [878148c266] - src: refactor IsSupportedAuthenticatedMode (Tobias Nießen) #42368
  • [970483ffd3] - src,crypto: handle empty maybe correctly in crypto_dh.cc (Darshan Sen) #42492
  • [a348f8ac1a] - src,crypto: remove uses of AllocatedBuffer from crypto_dh.cc (Darshan Sen) #42492
  • [fb25ba435c] - test: improve lib/internal/readline/promises.js coverage (MURAKAMI Masahiko) #42420
  • [4cbb1ea910] - test: remove hack for atob and btoa WPT tests (Antoine du Hamel) #42540
  • [f41a4780d5] - test: pass data into napi_create_external (Joyee Cheung) #42532
  • [90554572b5] - test: improve FileHandle.prototype.write coverage (Antoine du Hamel) #42541
  • [797994e4c0] - test: add test for exception handlings in debugger (Kohei Ueno) #42327
  • [8c9b5e9a36] - test: fix typo in common/wpt.js (Ikko Ashimine) #42567
  • [2f682091cd] - test: fix typos in test/parallel (Daeyeon Jeong) #42502
  • [52d1c8d6d9] - test: add trace-gc flag test (Tony Gorez) #42471
  • [19c933c79a] - test,fs: add fs.rm() tests for .git directories (Darshan Sen) #42410
  • [d64c4fb94d] - tools: enable no-var ESLint rule for lib (Rich Trott) #42573
  • [a9f2636d12] - tools: fixed bug causing JSON format to be broken (mawaregetsuka) #41565
  • [bff9cae16a] - tools: update GHA actions version (Antoine du Hamel) #42498
  • [c6bfb225cd] - tools: update eslint to 8.12.0 (Node.js GitHub Bot) #42489
  • [464e57ffc7] - tools: update lint-md-dependencies to vfile-reporter@7.0.4 (Node.js GitHub Bot) #42487
  • [13dd8e73df] - tools: refloat 7 Node.js patches to cpplint.py (Rich Trott) #42416
  • [3a1b0e5b87] - tools: bump cpplint to 1.6.0 (Rich Trott) #42416
  • [9344a06d9c] - tools: fix skip PR if CI is still running (Xuguang Mei) #42377

nodejs-github-bot commented Apr 5, 2022

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/actions

nodejs-github-bot added build Issues and PRs related to build files or the CI. dependencies Pull requests that update a dependency file. doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project. needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. v17.x labels Apr 5, 2022
juanarbol mentioned this pull request Apr 5, 2022
Closed

This comment was marked as outdated.

Sign in to view
juanarbol added a commit that referenced this pull request Apr 5, 2022
2022-04-04, Version 17.8.1 (Current), @BethGriggs prepared by @juanarbol
b23be87 


Notable Changes:

* doc: add @meixg to collaborators (Xuguang Mei) #42576

PR-URL: #42613

This comment was marked as outdated.

Sign in to view

nodejs-github-bot commented Apr 5, 2022

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/43338/

targos commented Apr 5, 2022

Copy link
Copy Markdown
Member

Commit message should be changed from 2022-04-04, Version 17.8.1 (Current), @BethGriggs prepared by @juanarbol to 2022-04-05, Version 17.9.0 (Current)

targos reviewed Apr 5, 2022
View reviewed changes
Comment thread doc/api/fs.md Outdated
targos previously requested changes Apr 5, 2022
View reviewed changes

targos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

there seems to be a semver-major change

aduh95 reviewed Apr 5, 2022
View reviewed changes
Comment thread doc/changelogs/CHANGELOG_V17.md Outdated
VoltrexKeyva added release Issues and PRs related to Node.js releases. and removed doc Issues and PRs related to the documentations. build Issues and PRs related to build files or the CI. meta Issues and PRs related to the general management of the project. tools Issues and PRs related to the tools directory. dependencies Pull requests that update a dependency file. labels Apr 5, 2022
BethGriggs requested changes Apr 5, 2022
View reviewed changes

BethGriggs left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

In addition to the other’s comments, we'll need to update the target release date in the changelog and the commit message

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view
2022-04-07, Version 17.9.0 (Current)
70dcef5 
Notable Changes:

* (SEMVER-MINOR) crypto: make authTagLength optional for CC20P1305 (Tobias Nießen) #42427
* deps: update undici to 4.16.0 (Node.js GitHub Bot) #42414
* doc: add @meixg to collaborators (Xuguang Mei) #42576

PR-URL: #42613

This comment was marked as outdated.

Sign in to view

juanarbol commented Apr 7, 2022 •
edited by BethGriggs
Loading

Copy link
Copy Markdown
Member Author

V8-CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/4613/
V8-CI rerun: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=rhel7-s390x,v8test=v8test/4616 ✅

  • I've been informed this is a known flake

CITGM-CI: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/
🤞

This comment was marked as outdated.

Sign in to view

nodejs-github-bot commented Apr 7, 2022 •
edited by BethGriggs
Loading

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/43373/
(all passed except ARM - likely flake)

This comment was marked as outdated.

Sign in to view

BethGriggs commented Apr 7, 2022

Copy link
Copy Markdown
Member

One more resume CI...we've been unlucky hitting machine disconnects:

21:35:32 Caused: hudson.remoting.ChannelClosedException: Channel "hudson.remoting.Channel@1c1cafe6:JNLP4-connect connection from 139.178.85.13/139.178.85.13:49554": Remote call on JNLP4-connect connection from 139.178.85.13/139.178.85.13:49554 failed. The channel is closing down or has closed down

nodejs-github-bot commented Apr 7, 2022 •
edited by BethGriggs
Loading

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/43377/ ✅
V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/4617/ ✅

BethGriggs changed the title V17.9.0 proposal v17.9.0 proposal Apr 7, 2022

BethGriggs commented Apr 7, 2022

Copy link
Copy Markdown
Member

Release build on 70dcef5: https://ci-release.nodejs.org/job/iojs+release/8375
Rerun for macOS pkg: https://ci-release.nodejs.org/job/iojs+release/8376

BethGriggs merged commit 70dcef5 into v17.x Apr 7, 2022
BethGriggs added a commit that referenced this pull request Apr 7, 2022
Working on v17.9.1
36ad9e8 
PR-URL: #42613
juanarbol added a commit that referenced this pull request Apr 7, 2022
2022-04-07, Version 17.9.0 (Current)
3c75264 
Notable Changes:

* (SEMVER-MINOR) crypto: make authTagLength optional for CC20P1305 (Tobias Nießen) #42427
* deps: update undici to 4.16.0 (Node.js GitHub Bot) #42414
* doc: add @meixg to collaborators (Xuguang Mei) #42576

PR-URL: #42613
juanarbol added a commit to juanarbol/nodejs.org that referenced this pull request Apr 7, 2022
Blog: v17.9.0 release post
afd1e08 
Refs: nodejs/node#42613
juanarbol mentioned this pull request Apr 7, 2022
Merged
BethGriggs pushed a commit to nodejs/nodejs.org that referenced this pull request Apr 7, 2022
Blog: v17.9.0 release post (#4533)
0d13fa7 
Refs: nodejs/node#42613
juanarbol deleted the v17.9.0-proposal branch April 7, 2022 23:05
This was referenced Apr 8, 2022
Open
Open
Open
Open
Open
Open
xtx1130 pushed a commit to xtx1130/node that referenced this pull request Apr 25, 2022
2022-04-07, Version 17.9.0 (Current)
765fc57 
Notable Changes:

* (SEMVER-MINOR) crypto: make authTagLength optional for CC20P1305 (Tobias Nießen) nodejs#42427
* deps: update undici to 4.16.0 (Node.js GitHub Bot) nodejs#42414
* doc: add @meixg to collaborators (Xuguang Mei) nodejs#42576

PR-URL: nodejs#42613
This file contains hidden or 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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

richardlau richardlau approved these changes
BethGriggs BethGriggs approved these changes
aduh95 aduh95 approved these changes
bengl Awaiting requested review from bengl
+1 more reviewer
targos targos left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

needs-ci PRs that need a full CI run. release Issues and PRs related to Node.js releases.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Back | FazBrowse Home | New Git URL