V22.22.1 proposal by aduh95 · Pull Request #61947 · nodejs/node
labels
aduh95 added a commit that referenced this pull request
Notable changes: build: * test on Python 3.14 (Christian Clauss) #59983 cli: * mark `--heapsnapshot-near-heap-limit` as stable (Joyee Cheung) #60956 crypto: * update root certificates to NSS 3.119 (Node.js GitHub Bot) #61419 * update root certificates to NSS 3.117 (Node.js GitHub Bot) #60741 doc: * add avivkeller to collaborators (Aviv Keller) #61115 * add gurgunday to collaborators (Gürgün Dayıoğlu) #61094 meta: * add Renegade334 to collaborators (Renegade334) #60714 PR-URL: #61947
This means that `node.h` can include only this file, instead of the entirety of `node_api.h`. Split out from #60496 since it was rightfully pointed out that the breaking part of the change should not touch Node-API headers. PR-URL: #60512 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
PR-URL: #60520 Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
PR-URL: #60422 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
PR-URL: #60348 Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com> Reviewed-By: Jason Zhang <xzha4350@gmail.com>
Bumps [actions/setup-node](https://github.com/actions/setup-node) from 5.0.0 to 6.0.0. - [Release notes](https://github.com/actions/setup-node/releases) - [Commits](actions/setup-node@a0853c2...2028fbc) --- updated-dependencies: - dependency-name: actions/setup-node dependency-version: 6.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> PR-URL: #60529 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
PR-URL: #60541 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
PR-URL: #60542 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
PR-URL: #60385 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
PR-URL: #59354 Reviewed-By: Tim Perry <pimterry@gmail.com> Reviewed-By: theanarkh <theratliter@gmail.com> Reviewed-By: Ethan Arrowood <ethan@arrowood.dev>
Previously when the child process helpers are used to print
information about the failed expectations and the env of the
child process was overridden, it printed the entire env object,
which may be too much if the caller does something like
{ env: { ENV: 'var', ...process.env } } (which tend to be always
the case for specifying env because we need to copy the
process.env for dynamic library loading in the CI).
This updates it to only show the env vars that differ from
process.env for a cleaner log in the case of failure.
PR-URL: #60556
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
This allows us to set up fixtures for the benchmark suite only once, which can save quite a bit of time when running benchmarks that require tens of thousands of fixture files or more (e.g. the module benchmarks). PR-URL: #60574 Fixes: #58488 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
PR-URL: #53864 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
`node:http`' `request.reusedSocket` documentation sample code was referencing an undeclared `agent` identifier. PR-URL: #55478 Reviewed-By: Ethan Arrowood <ethan@arrowood.dev> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
PR-URL: #61921 Reviewed-By: Stewart X Addison <sxa@redhat.com> Reviewed-By: Richard Lau <richard.lau@ibm.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
PR-URL: #61789 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Original commit message:
[riscv] Fix sp handling in MacroAssembler::LeaveFrame
Keep sp <= fp to ensure that data right above fp doesn't get clobbered
by an inopportune signal and its handler.
Such clobbering can happen in e.g. Node.js when JIT-compiled code is
interrupted by a SIGCHLD handler.
Bug: None
Change-Id: Ief0836032ada7942e89f081f7605f61632c4d414
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/7540554
Reviewed-by: Ji Qiu <qiuji@iscas.ac.cn>
Commit-Queue: Yahan Lu (LuYahan) <yahan@iscas.ac.cn>
Reviewed-by: Rezvan Mahdavi Hezaveh <rezvan@chromium.org>
Cr-Commit-Position: refs/heads/main@{#105069}
Refs: v8/v8@6a0a25a
Co-authored-by: kxxt <rsworktech@outlook.com>
PR-URL: #61688
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
aduh95 added a commit that referenced this pull request
Notable changes: build: * test on Python 3.14 (Christian Clauss) #59983 cli: * mark `--heapsnapshot-near-heap-limit` as stable (Joyee Cheung) #60956 crypto: * update root certificates to NSS 3.119 (Node.js GitHub Bot) #61419 * update root certificates to NSS 3.117 (Node.js GitHub Bot) #60741 doc: * add avivkeller to collaborators (Aviv Keller) #61115 * add gurgunday to collaborators (Gürgün Dayıoğlu) #61094 meta: * add Renegade334 to collaborators (Renegade334) #60714 PR-URL: #61947
aduh95 added a commit to aduh95/node that referenced this pull request
Notable changes: build: * test on Python 3.14 (Christian Clauss) nodejs#59983 cli: * mark `--heapsnapshot-near-heap-limit` as stable (Joyee Cheung) nodejs#60956 crypto: * update root certificates to NSS 3.119 (Node.js GitHub Bot) nodejs#61419 * update root certificates to NSS 3.117 (Node.js GitHub Bot) nodejs#60741 doc: * add avivkeller to collaborators (Aviv Keller) nodejs#61115 * add gurgunday to collaborators (Gürgün Dayıoğlu) nodejs#61094 meta: * add Renegade334 to collaborators (Renegade334) nodejs#60714 PR-URL: nodejs#61947
`parallel/test-strace-openat-openssl` was added to check explicitly
for a list of known files that would be opened for a set workload
(`require("crypto")`). This is not reliable when Node.js is linked
to an external/shared OpenSSL library (e.g. it might be configured
to load configuration files from a different default location and/or
load more than one configuration file) so skip this test when Node.js
is built in that way.
PR-URL: #61987
Fixes: #61966
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
aduh95 added a commit that referenced this pull request
build: * test on Python 3.14 (Christian Clauss) #59983 cli: * mark `--heapsnapshot-near-heap-limit` as stable (Joyee Cheung) #60956 crypto: * update root certificates to NSS 3.119 (Node.js GitHub Bot) #61419 * update root certificates to NSS 3.117 (Node.js GitHub Bot) #60741 doc: * add avivkeller to collaborators (Aviv Keller) #61115 * add gurgunday to collaborators (Gürgün Dayıoğlu) #61094 meta: * add Renegade334 to collaborators (Renegade334) #60714 PR-URL: #61947
PR-URL: #61732 Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
PR-URL: #61830 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com> Reviewed-By: Richard Lau <richard.lau@ibm.com>
PR-URL: #62016 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Richard Lau <richard.lau@ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Gürgün Dayıoğlu <hey@gurgun.day>
aduh95 added a commit that referenced this pull request
build: * test on Python 3.14 (Christian Clauss) #59983 cli: * mark `--heapsnapshot-near-heap-limit` as stable (Joyee Cheung) #60956 crypto: * update root certificates to NSS 3.119 (Node.js GitHub Bot) #61419 * update root certificates to NSS 3.117 (Node.js GitHub Bot) #60741 doc: * add avivkeller to collaborators (Aviv Keller) #61115 * add gurgunday to collaborators (Gürgün Dayıoğlu) #61094 meta: * add Renegade334 to collaborators (Renegade334) #60714 PR-URL: #61947
build: * test on Python 3.14 (Christian Clauss) #59983 cli: * mark `--heapsnapshot-near-heap-limit` as stable (Joyee Cheung) #60956 crypto: * update root certificates to NSS 3.119 (Node.js GitHub Bot) #61419 * update root certificates to NSS 3.117 (Node.js GitHub Bot) #60741 doc: * add avivkeller to collaborators (Aviv Keller) #61115 * add gurgunday to collaborators (Gürgün Dayıoğlu) #61094 meta: * add Renegade334 to collaborators (Renegade334) #60714 PR-URL: #61947
marco-ippolito pushed a commit that referenced this pull request
build: * test on Python 3.14 (Christian Clauss) #59983 cli: * mark `--heapsnapshot-near-heap-limit` as stable (Joyee Cheung) #60956 crypto: * update root certificates to NSS 3.119 (Node.js GitHub Bot) #61419 * update root certificates to NSS 3.117 (Node.js GitHub Bot) #60741 doc: * add avivkeller to collaborators (Aviv Keller) #61115 * add gurgunday to collaborators (Gürgün Dayıoğlu) #61094 meta: * add Renegade334 to collaborators (Renegade334) #60714 PR-URL: #61947
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