GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — v22.9.0 proposal by RafaelGSS · Pull Request #54966 · nodejs/node

Conversation

@RafaelGSS

@RafaelGSS RafaelGSS commented Sep 16, 2024 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

2024-09-17, Version 22.9.0 (Current), @RafaelGSS

New API to retrieve execution Stack Trace

A new API getCallSite has been introduced to the util module. This API allows users
to retrieve the stacktrace of the current execution. Example:

const util = require('node:util');

function exampleFunction() {
  const callSites = util.getCallSite();

  console.log('Call Sites:');
  callSites.forEach((callSite, index) => {
    console.log(`CallSite ${index + 1}:`);
    console.log(`Function Name: ${callSite.functionName}`);
    console.log(`Script Name: ${callSite.scriptName}`);
    console.log(`Line Number: ${callSite.lineNumber}`);
    console.log(`Column Number: ${callSite.column}`);
  });
  // CallSite 1:
  // Function Name: exampleFunction
  // Script Name: /home/example.js
  // Line Number: 5
  // Column Number: 26

  // CallSite 2:
  // Function Name: anotherFunction
  // Script Name: /home/example.js
  // Line Number: 22
  // Column Number: 3

  // ...
}

// A function to simulate another stack layer
function anotherFunction() {
  exampleFunction();
}

anotherFunction();

Thanks to Rafael Gonzaga for making this work on #54380.

Disable V8 Maglev

We have seen several crashes/unexpected JS behaviors with maglev on v22
(which ships V8 v12.4). The bugs lie in the codegen so it would be difficult for
users to work around them or even figure out where the bugs are coming from.
Some bugs are fixed in the upstream while some others probably remain.

As v22 will get stuck with V8 v12.4 as LTS, it will be increasingly difficult to
backport patches for them even if the bugs are fixed. So disable it by default
on v22 to reduce the churn and troubles for users.

Thanks to Joyee Cheung for making this work on #54384

Exposes X509_V_FLAG_PARTIAL_CHAIN to tls.createSecureContext

This releases introduces a new option to the API tls.createSecureContext. For
now on users can use tls.createSecureContext({ allowPartialTrustChain: true })
to treat intermediate (non-self-signed) certificates in the trust CA certificate
list as trusted.

Thanks to Anna Henningsen for making this work on #54790

Other Notable Changes

  • [5c9599af5a] - src: create handle scope in FastInternalModuleStat (Joyee Cheung) #54384
  • [e2307d87e8] - (SEMVER-MINOR) stream: relocate the status checking code in the onwritecomplete (YoonSoo_Shin) #54032

Deprecations

  • [8433032948] - repl: doc-deprecate instantiating node:repl classes without new (Aviv Keller) #54842
  • [8c4c85cf31] - zlib: deprecate instantiating classes without new (Yagiz Nizipli) #54708

Commits

  • [027b0ffe84] - async_hooks: add an InactiveAsyncContextFrame class (Bryan English) #54510
  • [022767028e] - benchmark: --no-warnings to avoid DEP/ExpWarn log (Rafael Gonzaga) #54928
  • [af1988c147] - benchmark: add buffer.isAscii benchmark (RafaelGSS) #54740
  • [40c6849964] - benchmark: add buffer.isUtf8 bench (RafaelGSS) #54740
  • [237d7dfbde] - benchmark: add access async version to bench (Rafael Gonzaga) #54747
  • [ebe91db827] - benchmark: enhance dc publish benchmark (Rafael Gonzaga) #54745
  • [060164485b] - benchmark: add match and doesNotMatch bench (RafaelGSS) #54734
  • [2844180c7e] - benchmark: add rejects and doesNotReject bench (RafaelGSS) #54734
  • [af7689ed02] - benchmark: add throws and doesNotThrow bench (RafaelGSS) #54734
  • [456a1fe222] - benchmark: add strictEqual and notStrictEqual bench (RafaelGSS) #54734
  • [721c63c858] - benchmark: adds groups to better separate benchmarks (Giovanni Bucci) #54393
  • [68e45b406e] - benchmark,doc: add CPU scaling governor to perf (Rafael Gonzaga) #54723
  • [d19efd7a50] - benchmark,doc: mention bar.R to the list of scripts (Rafael Gonzaga) #54722
  • [1fb67afa2f] - buffer: fix out of range for toString (Jason Zhang) #54553
  • [85b5ed5d41] - buffer: re-enable Fast API for Buffer.write (Robert Nagy) #54526
  • [9a075279ec] - build: upgrade clang-format to v18 (Aviv Keller) #53957
  • [69ec9d8d2b] - build: fix conflicting V8 object print flags (Daeyeon Jeong) #54785
  • [948bba396c] - build: do not build with code cache for core coverage collection (Joyee Cheung) #54633
  • [6200cf4fb6] - build: don't store eslint locally (Aviv Keller) #54231
  • [3b5ed97fe9] - build: turn off -Wrestrict (Richard Lau) #54737
  • [e38e305a35] - build,win: enable clang-cl compilation (Stefan Stojanovic) #54655
  • [5bba0781b0] - crypto: reject dh,x25519,x448 in {Sign,Verify}Final (Huáng Jùnliàng) #53774
  • [3981853c00] - crypto: return a clearer error when loading an unsupported pkcs12 (Tim Perry) #54485
  • [02ac5376b9] - crypto: remove unused kHashTypes internal (Antoine du Hamel) #54627
  • [323d9da3c9] - deps: update cjs-module-lexer to 1.4.1 (Node.js GitHub Bot) #54846
  • [bf4bf7cc6b] - deps: update simdutf to 5.5.0 (Node.js GitHub Bot) #54434
  • [61047dd130] - deps: upgrade npm to 10.8.3 (npm team) #54619
  • [2351da5034] - deps: update cjs-module-lexer to 1.4.0 (Node.js GitHub Bot) #54713
  • [0659516823] - deps: allow amaro to be externalizable (Michael Dawson) #54646
  • [6a32645dbc] - deps: fix sign-compare warning in ncrypto (Cheng) #54624
  • [8f62f19197] - doc: fix broken Android building link (Niklas Wenzel) #54922
  • [440c256d76] - doc: add support link for aduh95 (Antoine du Hamel) #54866
  • [56aca2a1ca] - doc: run license-builder (github-actions[bot]) #54854
  • [8931f569c6] - doc: experimental flag for global accessible APIs (Chengzhong Wu) #54330
  • [6f8a6e9eb6] - doc: add ERR_INVALID_ADDRESS to errors.md (Aviv Keller) #54661
  • [c1b92e05e7] - doc: add support link for mcollina (Matteo Collina) #54786
  • [1def18122a] - doc: mark --conditions CLI flag as stable (Guy Bedford) #54209
  • [b8ae36b6c3] - doc: fix typo in recognizing-contributors (Tobias Nießen) #54822
  • [2c2ae80924] - doc: clarify --max-old-space-size and --max-semi-space-size units (Alexandre ABRIOUX) #54477
  • [5bd4be5ce7] - doc: replace --allow-fs-read by --allow-fs-write in related section (M1CK431) #54427
  • [c0f3e4603f] - doc: add support link for marco-ippolito (Marco Ippolito) #54789
  • [dc69eb8276] - doc: fix typo in module.md (Tobias Nießen) #54794
  • [de225f5db9] - doc: specify that preloaded modules affect subprocesses (Aviv Keller) #52939
  • [62b0007cbe] - doc: clarify expandedSQL behavior (Tobias Nießen) #54685
  • [1c7bdf95db] - doc: render type references in SQLite docs (Tobias Nießen) #54684
  • [5555095531] - doc: fix typo (Michael Dawson) #54640
  • [754baa4efa] - doc: fix webcrypto.md AES-GCM backticks (Filip Skokan) #54621
  • [5bfb4bcf45] - doc: add documentation about os.tmpdir() overrides (Joyee Cheung) #54613
  • [22d873208e] - doc, build: fixup build docs (Aviv Keller) #54899
  • [5e081a12b6] - doc, child_process: add esm snippets (Aviv Keller) #53616
  • [2b68c30a26] - doc, meta: fix broken link in onboarding.md (Aviv Keller) #54886
  • [a624002fff] - esm: throw ERR_REQUIRE_ESM instead of ERR_INTERNAL_ASSERTION (Antoine du Hamel) #54868
  • [31d4ef91ee] - esm: fix support for URL instances in import.meta.resolve (Antoine du Hamel) #54690
  • [40ba89e452] - esm: use Undici/fetch data: URL parser (Matthew Aitken) #54748
  • [93116dd7b1] - fs: translate error code properly in cpSync (Jason Zhang) #54906
  • [375cbb592e] - fs: refactor rimraf to avoid using primordials (Yagiz Nizipli) #54834
  • [ee89c3149e] - fs: respect dereference when copy symlink directory (Jason Zhang) #54732
  • [7123bf7ca4] - http: reduce likelihood of race conditions on keep-alive timeout (jazelly) #54863
  • [04ef3e4afd] - https: only use default ALPNProtocols when appropriate (Brian White) #54411
  • [dc5593ba1e] - lib: remove unnecessary async (jakecastelli) #54829
  • [2b9a6373da] - lib: make WeakRef safe in abort_controller (jazelly) #54791
  • [5f02e1b850] - lib: move Symbol[Async]Dispose polyfills to internal/util (Antoine du Hamel) #54853
  • [fc78ced7e4] - lib: convert signals to array before validation (Jason Zhang) #54714
  • [21fef34a53] - lib: add note about removing node:sys module (Rafael Gonzaga) #54743
  • [a37d805489] - (SEMVER-MINOR) lib: add util.getCallSite() API (Rafael Gonzaga) #54380
  • [2a1f56cce6] - lib: ensure no holey array in fixed_queue (Jason Zhang) #54537
  • [540b1dbaf6] - lib: refactor SubtleCrypto experimental warnings (Filip Skokan) #54620
  • [b59c8b88c7] - lib,src: use built-in array buffer detach, transfer (Yagiz Nizipli) #54837
  • [c1cc046de9] - meta: bump peter-evans/create-pull-request from 6.1.0 to 7.0.1 (dependabot[bot]) #54820
  • [82c08ef483] - meta: add Windows ARM64 to flaky-tests list (Aviv Keller) #54693
  • [df30e8efa1] - meta: ping @nodejs/performance on bench changes (Rafael Gonzaga) #54752
  • [bdd9fbb905] - meta: bump actions/setup-python from 5.1.1 to 5.2.0 (Rich Trott) #54691
  • [19574a8403] - meta: update sccache to v0.8.1 (Aviv Keller) #54720
  • [9ebcfb2b28] - meta: bump step-security/harden-runner from 2.9.0 to 2.9.1 (dependabot[bot]) #54704
  • [ea58feb959] - meta: bump actions/upload-artifact from 4.3.4 to 4.4.0 (dependabot[bot]) #54703
  • [c6bd9e443e] - meta: bump github/codeql-action from 3.25.15 to 3.26.6 (dependabot[bot]) #54702
  • [79b358af2e] - meta: fix links in SECURITY.md (Aviv Keller) #54696
  • [6c8a20d650] - meta: fix contributing codeowners (Aviv Keller) #54641
  • [b7284ed099] - module: do not warn for typeless package.json when there isn't one (Joyee Cheung) #54045
  • [ddd24a6e63] - node-api: add external buffer creation benchmark (Chengzhong Wu) #54877
  • [4a7576efae] - node-api: add support for UTF-8 and Latin-1 property keys (Mert Can Altin) #52984
  • [461e523498] - os: improve tmpdir performance (Yagiz Nizipli) #54709
  • [94fb7ab2e7] - path: remove StringPrototypeCharCodeAt from posix.extname (Aviv Keller) #54546
  • [67b1d4cb45] - repl: avoid interpreting 'npm' as a command when errors are recoverable (Shima Ryuhei) #54848
  • [8433032948] - repl: doc-deprecate instantiating node:repl classes without new (Aviv Keller) #54842
  • [7766349dd0] - sqlite: fix segfault in expandedSQL (Tobias Nießen) #54687
  • [4c1b98ba2b] - sqlite: remove unnecessary auto assignment (Tobias Nießen) #54686
  • [77d162adb6] - src: add --env-file-if-exists flag (Bosco Domingo) #53060
  • [424bdc03b4] - src: add Cleanable class to Environment (Gabriel Schulhof) #54880
  • [fbd08e3a9f] - src: switch crypto APIs to use Maybe<void> (James M Snell) #54775
  • [5e72bd3545] - src: eliminate ManagedEVPPkey (James M Snell) #54751
  • [97cbcfbb43] - src: fix unhandled error in structuredClone (Daeyeon Jeong) #54764
  • [b89cd8d19a] - src: move hkdf, scrypto, pbkdf2 impl to ncrypto (James M Snell) #54651
  • [5c9599af5a] - src: create handle scope in FastInternalModuleStat (Joyee Cheung) #54384
  • [e2307d87e8] - (SEMVER-MINOR) stream: relocate the status checking code in the onwritecomplete (YoonSoo_Shin) #54032
  • [ff54cabef6] - test: adjust test-tls-junk-server for OpenSSL32 (Michael Dawson) #54926
  • [23fb03beed] - test: remove duplicate skip AIX (Wuli) #54917
  • [2b5e70816a] - test: adjust tls test for OpenSSL32 (Michael Dawson) #54909
  • [cefa692dcb] - test: fix test-http2-socket-close.js (Hüseyin Açacak) #54900
  • [097f6d3e7e] - test: improve test-internal-fs-syncwritestream (Sunghoon) #54671
  • [ed736a689f] - test: deflake test-dns (Luigi Pinca) #54902
  • [bb4849f595] - test: fix test test-tls-dhe for OpenSSL32 (Michael Dawson) #54903
  • [d9264bceca] - test: use correct file naming syntax for util-parse-env (Aviv Keller) #53705
  • [115a7ca42a] - test: add missing await (Luigi Pinca) #54828
  • [7a1d633d77] - test: move more url tests to node:test (Yagiz Nizipli) #54636
  • [ee385d62b9] - test: strip color chars in test-runner-run (Giovanni Bucci) #54552
  • [2efec6221c] - test: deflake test-http2-misbehaving-multiplex (Luigi Pinca) #54872
  • [b198a91404] - test: remove dead code in test-http2-misbehaving-multiplex (Luigi Pinca) #54860
  • [194cb83f39] - test: reduce test-esm-loader-hooks-inspect-wait flakiness (Luigi Pinca) #54827
  • [4b53558e8b] - test: reduce the allocation size in test-worker-arraybuffer-zerofill (James M Snell) #54839
  • [c968d65d6d] - test: fix test-tls-client-mindhsize for OpenSSL32 (Michael Dawson) #54739
  • [b998bb0933] - test: remove need to make fs call for zlib test (Yagiz Nizipli) #54814
  • [f084ea2e01] - test: use platform timeout (jakecastelli) #54591
  • [b10e434cf3] - test: add platform timeout support for riscv64 (jakecastelli) #54591
  • [b875f2d7de] - test: reduce stack size for test-error-serdes (James M Snell) #54840
  • [d1a411480a] - test: reduce fs calls in test-fs-existssync-false (Yagiz Nizipli) #54815
  • [b96ee30a09] - test: use node:test in test-cli-syntax.bad (Aviv Keller) #54513
  • [5278b8b7a1] - test: move test-http-server-request-timeouts-mixed (James M Snell) #54841
  • [8345a60d3a] - test: fix Windows async-context-frame memory failure (Stephen Belanger) #54823
  • [cad404e1a1] - test: fix volatile for CauseSegfault with clang (Ivan Trubach) #54325
  • [41682c7286] - test: set test-http2-socket-close as flaky (Yagiz Nizipli) #54802
  • [1e1ac48711] - test: set test-worker-arraybuffer-zerofill as flaky (Yagiz Nizipli) #54802
  • [56238debff] - test: set test-runner-run-watch as flaky (Yagiz Nizipli) #54802
  • [8291de1540] - test: set test-http-server-request-timeouts-mixed as flaky (Yagiz Nizipli) #54802
  • [32d340e6b3] - test: set test-single-executable-application-empty as flaky (Yagiz Nizipli) #54802
  • [6a2da4c4ca] - test: set test-macos-app-sandbox as flaky (Yagiz Nizipli) #54802
  • [2f408847a0] - test: set test-fs-utimes as flaky (Yagiz Nizipli) #54802
  • [e3b7c40ffc] - test: set test-runner-run-watch as flaky (Yagiz Nizipli) #54802
  • [d2ede46946] - test: set test-sqlite-statement-sync as flaky (Yagiz Nizipli) #54802
  • [b9f3385808] - test: set test-writewrap as flaky (Yagiz Nizipli) #54802
  • [d55fec8f40] - test: set test-async-context-frame as flaky (Yagiz Nizipli) #54802
  • [3dfb525f3e] - test: set test-esm-loader-hooks-inspect-wait as flaky (Yagiz Nizipli) #54802
  • [b0458a88b4] - test: set test-http2-large-file as flaky (Yagiz Nizipli) #54802
  • [5f6f8757e5] - test: set test-runner-watch-mode-complex as flaky (Yagiz Nizipli) #54802
  • [4231af336d] - test: set test-performance-function as flaky (Yagiz Nizipli) #54802
  • [45ef2a868e] - test: set test-debugger-heap-profiler as flaky (Yagiz Nizipli) #54802
  • [b5137f6405] - test: fix test-process-load-env-file when path contains ' (Antoine du Hamel) #54511
  • [960116905a] - test: refactor fs-watch tests due to macOS issue (Santiago Gimeno) #54498
  • [f074d74bf3] - test: refactor test-esm-type-field-errors (Giovanni Bucci) #54368
  • [67e30deced] - test: move more zlib tests to node:test (Yagiz Nizipli) #54609
  • [fdb65111a3] - test: improve output of child process utilities (Joyee Cheung) #54622
  • [55a12a4190] - test,crypto: update WebCryptoAPI WPT (Filip Skokan) #54925
  • [de0f445a7f] - test_runner: reimplement assert.ok to allow stack parsing (Aviv Keller) #54776
  • [a52c199d9d] - (SEMVER-MINOR) test_runner: report coverage thresholds in test:coverage (Aviv Keller) #54813
  • [6552fddef5] - test_runner: update kPatterns (Pietro Marchini) #54728
  • [3396a4954d] - test_runner: detect only tests when isolation is off (Colin Ihrig) #54832
  • [021f59b6bc] - test_runner: apply filtering when tests begin (Colin Ihrig) #54832
  • [36da793350] - test_runner: allow --import with no isolation (Aviv Keller) #54697
  • [de73d1ee4b] - test_runner: improve code coverage cleanup (Colin Ihrig) #54856
  • [3d478728f2] - timers: avoid generating holey internal arrays (Gürgün Dayıoğlu) #54771
  • [b3d567ae0f] - timers: document ref option for scheduler.wait (Paolo Insogna) #54605
  • [c2bf0134ce] - (SEMVER-MINOR) tls: add allowPartialTrustChain flag (Anna Henningsen) #54790
  • [608a611132] - tools: add readability/fn_size to filter (Rafael Gonzaga) #54744
  • [93fab49099] - tools: add util scripts to land and rebase PRs (Antoine du Hamel) #54656
  • [d6df542ff8] - tools: remove readability/fn_size rule (Rafael Gonzaga) #54663
  • [689d127ee7] - typings: fix TypedArray to a global type (1ilsang) #54063
  • [071dff1d34] - typings: correct param type of SafePromisePrototypeFinally (Wuli) #54727
  • [5243e3240c] - Revert "v8: enable maglev on supported architectures" (Joyee Cheung) #54384
  • [ade9da5b3a] - vm: add vm proto property lookup test (Chengzhong Wu) #54606
  • [8385958b60] - zlib: add typings for better dx (Yagiz Nizipli) #54699
  • [8c4c85cf31] - zlib: deprecate instantiating classes without new (Yagiz Nizipli) #54708

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

bricss reacted with thumbs up emoji bricss and Yovach reacted with hooray emoji bricss and Yovach reacted with heart emoji bricss and Yovach reacted with rocket emoji
All reactions
  • 👍 1 reaction
  • 🎉 2 reactions
  • ❤️ 2 reactions
  • 🚀 2 reactions
joyeecheung and others added 30 commits September 5, 2024 22:10
@joyeecheung @RafaelGSS
src: create handle scope in FastInternalModuleStat
5c9599a
It needs a handle scope for the context handle. Since the
FastApiCallbackOptions struct doesn't have isolate on it
in V8 12.4 on Node.js 22, use Isolate::TryGetCurrent() to
get to the isolate needed for the handle scope creation and
fallback to the slow callback if no isolate is entered.

PR-URL: #54384
Refs: #52797
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@joyeecheung @RafaelGSS
Revert "v8: enable maglev on supported architectures"
5243e32
This reverts commit 1a5acd0.

Reason to revert: we have seen several crashes/unexpected JS behaviors
with maglev on v22 (which ships V8 v12.4). The bugs lie in the codegen
so it would be difficult for users to work around them or even figure
out where the bugs are coming from. Some bugs are fixed in the upstream
while some others probably remain. As v22 will get stuck with V8 v12.4
as LTS, it will be increasingly difficult to backport patches for them
even if the bugs are fixed. So disable it by default on v22 to reduce
the churn and troubles for users.

PR-URL: #54384
Refs: #52797
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@joyeecheung @aduh95
module: do not warn for typeless package.json when there isn't one
b7284ed
It was intended that warnings should only be emitted for an
existing package.json without a type. This fixes a confusing
warning telling users to update /package.json when there are
no package.json on the lookup path at all, like this:

[MODULE_TYPELESS_PACKAGE_JSON] Warning: ... parsed as an ES module
because module syntax was detected; to avoid the performance penalty
of syntax detection, add "type": "module" to /package.json

Drive-by: update the warning message to be clear about
reparsing and make it clear what's actionable.

PR-URL: #54045
Reviewed-By: Geoffrey Booth <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@joyeecheung @aduh95
doc: add documentation about os.tmpdir() overrides
5bfb4bc
This documents the TMPDIR, TEMP and TMP overrides on different
platforms and that some operating systems set these by default.

PR-URL: #54613
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@bengl @aduh95
async_hooks: add an InactiveAsyncContextFrame class
027b0ff
This gives a class prototype for AsyncContextFrame that contains the
required methods, so that when we swap the prototype,
ActiveAsyncContextFrame methods are used instead. Previously, the
methods were defined in AsyncContextFrame, so swapping the prototype
didn't swap those static methods.

Also, make the ActiveAsyncContextFrame extend from Map.

Fixes: #54503
PR-URL: #54510
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Stephen Belanger <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Gerhard Stöbich <[email protected]>
@panva @aduh95
lib: refactor SubtleCrypto experimental warnings
540b1db
PR-URL: #54620
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
@panva @aduh95
doc: fix webcrypto.md AES-GCM backticks
754baa4
PR-URL: #54621
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@joyeecheung @aduh95
test: improve output of child process utilities
fdb6511
- Display command and options when it fails
- Keep the caller line at the top of the stack trace.

PR-URL: #54622
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Chemi Atlow <[email protected]>
@zcbenz @aduh95
deps: fix sign-compare warning in ncrypto
6a32645
PR-URL: #54624
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Chemi Atlow <[email protected]>
@aduh95
crypto: remove unused kHashTypes internal
02ac537
PR-URL: #54627
Reviewed-By: Filip Skokan <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@mhdawson @aduh95
doc: fix typo
5555095
name used instructions differed that what is in the code

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: #54640
Reviewed-By: Gerhard Stöbich <[email protected]>
Reviewed-By: Jake Yuesong Li <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@avivkeller @aduh95
meta: fix contributing codeowners
6c8a20d
PR-URL: #54641
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@anonrig @aduh95
test: move more zlib tests to node:test
67e30de
PR-URL: #54609
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Chemi Atlow <[email protected]>
@mscdex @aduh95
https: only use default ALPNProtocols when appropriate
04ef3e4
PR-URL: #54411
Reviewed-By: Robert Nagy <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Paolo Insogna <[email protected]>
Reviewed-By: Tim Perry <[email protected]>
Reviewed-By: Ethan Arrowood <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@RafaelGSS @aduh95
tools: remove readability/fn_size rule
d6df542
PR-URL: #54663
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@jazelly @jakecastelli @aduh95
lib: ensure no holey array in fixed_queue
2a1f56c
Co-authored-by: Jake Yuesong Li <[email protected]>
PR-URL: #54537
Fixes: #54472
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Jake Yuesong Li <[email protected]>
@ShogunPanda @aduh95
timers: document ref option for scheduler.wait
b3d567a
PR-URL: #54605
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jake Yuesong Li <[email protected]>
@MCprotein @aduh95
stream: relocate the status checking code in the onwritecomplete
e2307d8
relocate the status checking code before verifying if the stream is
destroyed

PR-URL: #54032
Reviewed-By: Robert Nagy <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@tniessen @aduh95
doc: render type references in SQLite docs
1c7bdf9
Render JavaScript types as such in the section "Type conversion between
JavaScript and SQLite".

PR-URL: #54684
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@iwuliz @aduh95
typings: correct param type of SafePromisePrototypeFinally
071dff1
PR-URL: #54727
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Jake Yuesong Li <[email protected]>
@avivkeller @aduh95
meta: fix links in SECURITY.md
79b358a
PR-URL: #54696
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
@dependabot @aduh95
meta: bump github/codeql-action from 3.25.15 to 3.26.6
c6bd9e4
Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.25.15 to 3.26.6.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@afb54ba...4dd1613)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
PR-URL: #54702
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@dependabot @aduh95
meta: bump actions/upload-artifact from 4.3.4 to 4.4.0
ea58feb
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4.3.4 to 4.4.0.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@0b2256b...5076954)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
PR-URL: #54703
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@dependabot @aduh95
meta: bump step-security/harden-runner from 2.9.0 to 2.9.1
9ebcfb2
Bumps [step-security/harden-runner](https://github.com/step-security/harden-runner) from 2.9.0 to 2.9.1.
- [Release notes](https://github.com/step-security/harden-runner/releases)
- [Commits](step-security/harden-runner@0d38121...5c7944e)

---
updated-dependencies:
- dependency-name: step-security/harden-runner
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
PR-URL: #54704
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Giovanni Bucci @Mesteery @aduh95
benchmark: adds groups to better separate benchmarks
721c63c
Fixes: #26425
Co-Authored-By: Yaman Kassir <[email protected]>
PR-URL: #54393
Reviewed-By: Vinícius Lourenço Claro Cardoso <[email protected]>
@richardlau @aduh95
build: turn off -Wrestrict
3b5ed97
PR-URL: #54737
Refs: #54736
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Jake Yuesong Li <[email protected]>
@legendecas @aduh95
vm: add vm proto property lookup test
ade9da5
Add more test coverage on vm prototype properties lookup with
`in` operator and property access.

PR-URL: #54606
Refs: #54436
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@avivkeller @aduh95
meta: update sccache to v0.8.1
19574a8
PR-URL: #54720
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@Trott @aduh95
meta: bump actions/setup-python from 5.1.1 to 5.2.0
bdd9fbb
PR-URL: #54691
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@RafaelGSS @aduh95
benchmark,doc: mention bar.R to the list of scripts
d19efd7
PR-URL: #54722
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
jakecastelli and others added 17 commits September 17, 2024 11:18
@jakecastelli @jazelly @RafaelGSS
lib: remove unnecessary async
dc5593b
Co-authored-by: Jason Zhang <[email protected]>
PR-URL: #54829
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
@nodejs-github-bot @RafaelGSS
deps: update cjs-module-lexer to 1.4.1
323d9da
PR-URL: #54846
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@anonrig @RafaelGSS
zlib: add typings for better dx
8385958
PR-URL: #54699
Reviewed-By: LiviaMedeiros <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@avivkeller @RafaelGSS
build: upgrade clang-format to v18
9a07527
PR-URL: #53957
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@huseyinacacak-janea @RafaelGSS
test: fix test-http2-socket-close.js
cefa692
Fixes: #54819
PR-URL: #54900
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
@mhdawson @RafaelGSS
test: adjust tls test for OpenSSL32
2b5e708
Refs: #53382

Looks like test is forcing an error through bad data and
the error code we get is different for OpenSSL32. Adjust
test to cope with the variation across versions.

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: #54909
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@iwuliz @RafaelGSS
test: remove duplicate skip AIX
23fb03b
PR-URL: #54917
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Xuguang Mei <[email protected]>
Reviewed-By: Jake Yuesong Li <[email protected]>
@nikwen @RafaelGSS
doc: fix broken Android building link
8f62f19
PR-URL: #54922
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Xuguang Mei <[email protected]>
@islandryu @RafaelGSS
repl: avoid interpreting 'npm' as a command when errors are recoverable
67b1d4c
This change ensures that 'npm' within JavaScript code is not mistakenly
interpreted as an npm command when the error is recoverable.
This allows 'npm' to be treated as expected in such scenarios.

Fixes: #54830
PR-URL: #54848
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Kohei Ueno <[email protected]>
@avivkeller @RafaelGSS
test_runner: reimplement assert.ok to allow stack parsing
de0f445
PR-URL: #54776
Refs: nodejs/help#4461
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Chemi Atlow <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
@panva @RafaelGSS
test,crypto: update WebCryptoAPI WPT
55a12a4
PR-URL: #54925
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
@mhdawson @RafaelGSS
test: adjust test-tls-junk-server for OpenSSL32
ff54cab
Refs: #53382

OpenSSL32 returns different error text. Looking through
the test it seems like the expected error text has been adjusted
for different OpenSSL versions in the past and what the test
is testing is not related to the error being returned.

Update test to allow for error returned by OpenSSL32

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: #54926
Refs: #53382
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@RafaelGSS
benchmark: --no-warnings to avoid DEP/ExpWarn log
0227670
PR-URL: #54928
Reviewed-By: Vinícius Lourenço Claro Cardoso <[email protected]>
Reviewed-By: Filip Skokan <[email protected]>
@anonrig @RafaelGSS
lib,src: use built-in array buffer detach, transfer
b59c8b8
PR-URL: #54837
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Stephen Belanger <[email protected]>
@BoscoDomingo @RafaelGSS
src: add --env-file-if-exists flag
77d162a
Fixes: #50993
Refs: #51451

test: remove unnecessary comment

src: conform to style guidelines

src: change flag to `--env-file-optional`

test: revert automatic linter changes

doc: fix typos

src: change flag to `--env-file-if-exists`

src: refactor `env_file_data` and `GetEnvFileDataFromArgs`

test: clean up tests

src: print error when file not found

test: remove unnecessary extras
PR-URL: #53060
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
@jazelly @jakecastelli @RafaelGSS
fs: translate error code properly in cpSync
93116dd
UV error code needs to be negative integer so it can be mapped
correctly. The filesystem error are positive integer, so we need to
handle it before throwing.

Co-authored-by: Jake Yuesong Li <[email protected]>
PR-URL: #54906
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
@RafaelGSS
2024-09-17, Version 22.9.0 (Current)
4631be0
Notable changes:

lib:
  * (SEMVER-MINOR) add util.getCallSite() API (Rafael Gonzaga) #54380
repl:
  * doc-deprecate instantiating `node:repl` classes without `new` (Aviv Keller) #54842
src:
  * create handle scope in FastInternalModuleStat (Joyee Cheung) #54384
stream:
  * (SEMVER-MINOR) relocate the status checking code in the onwritecomplete (YoonSoo_Shin) #54032
tls:
  * (SEMVER-MINOR) add `allowPartialTrustChain` flag (Anna Henningsen) #54790
v8:
  * Revert "v8: enable maglev on supported architectures (Joyee Cheung) #54384

PR-URL: #54966
@RafaelGSS
RafaelGSS force-pushed the v22.9.0-proposal branch from 9cc8a19 to 4631be0 Compare September 17, 2024 14:21
@RafaelGSS

RafaelGSS commented Sep 17, 2024 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member Author

CI: https://ci.nodejs.org/job/node-test-pull-request/62491/ -> https://ci.nodejs.org/job/node-test-pull-request/62496/ 🟢
CITGM: https://ci.nodejs.org/job/citgm-smoker/3475/ 🟡
📦 https://ci-release.nodejs.org/job/iojs+release/10476/ 🟢

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@RafaelGSS
RafaelGSS merged commit 4631be0 into v22.x Sep 17, 2024
@RafaelGSS
RafaelGSS deleted the v22.9.0-proposal branch September 17, 2024 20:50
RafaelGSS added a commit that referenced this pull request Sep 17, 2024
@RafaelGSS
Working on v22.9.1
8a3482e
PR-URL: #54966
RafaelGSS added a commit that referenced this pull request Sep 17, 2024
@RafaelGSS
2024-09-17, Version 22.9.0 (Current)
e49cf7a
Notable changes:

lib:
  * (SEMVER-MINOR) add util.getCallSite() API (Rafael Gonzaga) #54380
repl:
  * doc-deprecate instantiating `node:repl` classes without `new` (Aviv Keller) #54842
src:
  * create handle scope in FastInternalModuleStat (Joyee Cheung) #54384
stream:
  * (SEMVER-MINOR) relocate the status checking code in the onwritecomplete (YoonSoo_Shin) #54032
tls:
  * (SEMVER-MINOR) add `allowPartialTrustChain` flag (Anna Henningsen) #54790
v8:
  * Revert "v8: enable maglev on supported architectures (Joyee Cheung) #54384

PR-URL: #54966
RafaelGSS added a commit to nodejs/nodejs.org that referenced this pull request Sep 17, 2024
@RafaelGSS
Blog: v22.9.0 release post
cf7614a
Refs: nodejs/node#54966
RafaelGSS added a commit to nodejs/nodejs.org that referenced this pull request Sep 17, 2024
@RafaelGSS
Blog: v22.9.0 release post
1960941
Refs: nodejs/node#54966
@RafaelGSS RafaelGSS mentioned this pull request Sep 17, 2024
Blog: v22.9.0 release post nodejs/nodejs.org#7052
Merged
RafaelGSS added a commit to nodejs/nodejs.org that referenced this pull request Sep 17, 2024
@RafaelGSS
Blog: v22.9.0 release post (#7052)
b26f16e
Refs: nodejs/node#54966
joeeames pushed a commit to joeeames/nodejs.org that referenced this pull request Sep 24, 2024
@RafaelGSS @joeeames
Blog: v22.9.0 release post (nodejs#7052)
fc02826
Refs: nodejs/node#54966
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aduh95 aduh95 aduh95 left review comments
@avivkeller avivkeller avivkeller left review comments
@mcollina mcollina mcollina approved these changes
+1 more reviewer
@targos targos targos left review comments
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

needs-ci PRs that need a full CI run. release Issues and PRs related to Node.js releases. v22.x Issues that can be reproduced on v22.x or PRs targeting the v22.x-staging branch.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

20 participants

@RafaelGSS @nodejs-github-bot @avivkeller @anonrig @cjihrig @marco-ippolito @mcollina @targos @aduh95 @joyeecheung @bengl @panva @zcbenz @mhdawson @mscdex @jazelly @ShogunPanda @MCprotein @tniessen @iwuliz
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.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.