[v16.x backport] WPT runner update and crypto followups #44872

panva · 2022-10-03T09:12:44Z

Backports (in order):

PR-URL: nodejs#44196 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>

PR-URL: nodejs#44166 Reviewed-By: Feng Yu <[email protected]> Reviewed-By: Akhil Marsonya <[email protected]>

PR-URL: nodejs#43200 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Darshan Sen <[email protected]>

PR-URL: nodejs#44184 Reviewed-By: Feng Yu <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Akhil Marsonya <[email protected]> Reviewed-By: Erick Wendel <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]>

PR-URL: nodejs#44216 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Darshan Sen <[email protected]>

PR-URL: nodejs#44231 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Rich Trott <[email protected]>

Mark `test-http-pipeline-requests-connection-leak` flaky on IBM i. PR-URL: nodejs#44215 Refs: nodejs#43509 Refs: nodejs#39683 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Darshan Sen <[email protected]>

PR-URL: nodejs#44230 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>

PR-URL: nodejs#44229 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Feng Yu <[email protected]> Reviewed-By: Rich Trott <[email protected]>

Fixes: nodejs#44123 PR-URL: nodejs#44248 Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>

PR-URL: nodejs#44233 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: LiviaMedeiros <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Anto Aravinth <[email protected]> Reviewed-By: Feng Yu <[email protected]>

If the exception is handled by the userland process#uncaughtException handler, reports should not be generated repetitively as the process may continue to run. PR-URL: nodejs#44208 Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Stephen Belanger <[email protected]>

PR-URL: nodejs#44217 Reviewed-By: Robert Nagy <[email protected]> Reviewed-By: Matteo Collina <[email protected]>

PR-URL: nodejs#44244 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>

PR-URL: nodejs#43253 Reviewed-By: Erick Wendel <[email protected]>

Use the logical OR operator instead of the ternary operator where applicable. PR-URL: nodejs#44236 Reviewed-By: theanarkh <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Anto Aravinth <[email protected]> Reviewed-By: Feng Yu <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Darshan Sen <[email protected]>

PR-URL: nodejs#44243 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Darshan Sen <[email protected]>

Signed-off-by: Daeyeon Jeong [email protected] PR-URL: nodejs#44235 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Matteo Collina <[email protected]>

PR-URL: nodejs#43474 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Juan José Arboleda <[email protected]> Reviewed-By: Rich Trott <[email protected]>

PR-URL: nodejs#44263 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Myles Borins <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Rich Trott <[email protected]>

While copying BaseObjectPtrs is cheap, it is not a no-op, so it is best to iterate over references to BaseObjectPtrs only. PR-URL: nodejs#44270 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>

Filenames `stdout` and `stderr` have special meanings when writing the report. PR-URL: nodejs#44257 Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Erick Wendel <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Darshan Sen <[email protected]>

Extract common context embedder tag checks to ContextEmbedderTag so that verifying if a context is a node context doesn't need to access private members of node::Environment. PR-URL: nodejs#44258 Refs: nodejs#44179 Refs: nodejs#44252 Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Darshan Sen <[email protected]>

Add section with high level approach/requirements for enabling TypeScript execution as discussed in the Next-10 [TypeScript mini-summit](nodejs/next-10#150) Signed-off-by: Michael Dawson <[email protected]> PR-URL: nodejs#44030 Reviewed-By: Geoffrey Booth <[email protected]> Reviewed-By: Matteo Collina <[email protected]>

PR-URL: nodejs#44271 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>

KeyPairGenTraits::EncodeKey should not use an object that it previously moved. Make ManagedEVPPKey::ToEncoded(Public|Private)Key non-static members of ManagedEVPPKey and call them on the key object instead. PR-URL: nodejs#44269 Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Feng Yu <[email protected]>

PR-URL: nodejs#44262 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Kohei Ueno <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Harshitha K P <[email protected]> Reviewed-By: Jacob Smith <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>

The callback argument is required. PR-URL: nodejs#44311 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Moshe Atlow <[email protected]>

PR-URL: nodejs#44284 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]>

PR-URL: nodejs#44321 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>

PR-URL: #43455 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Backport-PR-URL: #44872

PR-URL: #44170 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: #44872

PR-URL: #44171 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: #44872

PR-URL: #44172 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: #44872

PR-URL: #44173 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: #44872

PR-URL: #44201 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: #44872

PR-URL: #44223 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Feng Yu <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Akhil Marsonya <[email protected]> Reviewed-By: LiviaMedeiros <[email protected]> Backport-PR-URL: #44872

We run all WPT from one subset in the same process using workers. As the number of the tests grow, it can take longer to run some of the subsets, but it's still overall faster than running them in different processes. This patch increases the timeout for WPT to prevent the test from failing because it takes longer to run (even though it would still complete at some point). Refs: nodejs/reliability#371 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> PR-URL: #44574 Backport-PR-URL: #44872

PR-URL: nodejs/node#43455 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Backport-PR-URL: nodejs/node#44872

PR-URL: nodejs/node#44170 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: nodejs/node#44872

PR-URL: nodejs/node#44171 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: nodejs/node#44872

PR-URL: nodejs/node#44172 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: nodejs/node#44872

PR-URL: nodejs/node#44173 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: nodejs/node#44872

PR-URL: nodejs/node#44201 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: nodejs/node#44872

PR-URL: nodejs/node#44223 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Feng Yu <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Akhil Marsonya <[email protected]> Reviewed-By: LiviaMedeiros <[email protected]> Backport-PR-URL: nodejs/node#44872

We run all WPT from one subset in the same process using workers. As the number of the tests grow, it can take longer to run some of the subsets, but it's still overall faster than running them in different processes. This patch increases the timeout for WPT to prevent the test from failing because it takes longer to run (even though it would still complete at some point). Refs: nodejs/reliability#371 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> PR-URL: nodejs/node#44574 Backport-PR-URL: nodejs/node#44872

PR-URL: nodejs/node#43455 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Backport-PR-URL: nodejs/node#44872

PR-URL: nodejs/node#44170 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: nodejs/node#44872

PR-URL: nodejs/node#44171 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: nodejs/node#44872

PR-URL: nodejs/node#44172 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: nodejs/node#44872

PR-URL: nodejs/node#44173 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: nodejs/node#44872

PR-URL: nodejs/node#44201 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: nodejs/node#44872

PR-URL: nodejs/node#44223 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Feng Yu <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Akhil Marsonya <[email protected]> Reviewed-By: LiviaMedeiros <[email protected]> Backport-PR-URL: nodejs/node#44872

We run all WPT from one subset in the same process using workers. As the number of the tests grow, it can take longer to run some of the subsets, but it's still overall faster than running them in different processes. This patch increases the timeout for WPT to prevent the test from failing because it takes longer to run (even though it would still complete at some point). Refs: nodejs/reliability#371 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> PR-URL: nodejs/node#44574 Backport-PR-URL: nodejs/node#44872

fossamagna and others added 30 commits September 23, 2022 11:49

test: use mustSucceed instead of mustCall with assert.ifError

38d92ab

PR-URL: nodejs#44196 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>

doc: improved building doc for Android

ccfdb47

PR-URL: nodejs#44166 Reviewed-By: Feng Yu <[email protected]> Reviewed-By: Akhil Marsonya <[email protected]>

build: enable pointer authentication for branch protection on arm64

68aaec5

PR-URL: nodejs#43200 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Darshan Sen <[email protected]>

src: fix --heapsnapshot-near-heap-limit error hint

1e3ea5f

PR-URL: nodejs#44216 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Darshan Sen <[email protected]>

meta: update AUTHORS

f2dc7c2

PR-URL: nodejs#44231 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Rich Trott <[email protected]>

test: mark connection leak test flaky on IBM i

1338afb

Mark `test-http-pipeline-requests-connection-leak` flaky on IBM i. PR-URL: nodejs#44215 Refs: nodejs#43509 Refs: nodejs#39683 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Darshan Sen <[email protected]>

tools: update lint-md-dependencies to [email protected]

89a3ed6

PR-URL: nodejs#44230 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>

deps: update corepack to 0.12.3

ab60ceb

PR-URL: nodejs#44229 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Feng Yu <[email protected]> Reviewed-By: Rich Trott <[email protected]>

doc: add cola119 to collaborators

1bc2090

Fixes: nodejs#44123 PR-URL: nodejs#44248 Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>

http: add max for http keepalive

c6c77d0

PR-URL: nodejs#44217 Reviewed-By: Robert Nagy <[email protected]> Reviewed-By: Matteo Collina <[email protected]>

tools: update lint-md-dependencies to [email protected]

57eb268

PR-URL: nodejs#44244 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>

trace_events: add example

993cf51

PR-URL: nodejs#43253 Reviewed-By: Erick Wendel <[email protected]>

tools: update ESLint to 8.22.0

0759337

PR-URL: nodejs#44243 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Darshan Sen <[email protected]>

meta: update web streams in label-pr-config

00bc140

Signed-off-by: Daeyeon Jeong [email protected] PR-URL: nodejs#44235 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Matteo Collina <[email protected]>

lib: refactor to avoid prototype pollution

807a659

PR-URL: nodejs#43474 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Juan José Arboleda <[email protected]> Reviewed-By: Rich Trott <[email protected]>

deps: upgrade npm to 8.18.0

bb91f62

PR-URL: nodejs#44263 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Myles Borins <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Rich Trott <[email protected]>

doc: fix optionality of callback arg of checkPrime

ce1a3e2

The callback argument is required. PR-URL: nodejs#44311 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Moshe Atlow <[email protected]>

test_runner: fix test runner hooks failure stack

2b9e88d

PR-URL: nodejs#44284 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]>

meta: update AUTHORS

34d4fb2

PR-URL: nodejs#44321 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>

juanarbol closed this Oct 11, 2022

juanarbol pushed a commit that referenced this pull request Oct 11, 2022

test: update WPT runner

14c7f4a

PR-URL: #43455 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Backport-PR-URL: #44872

juanarbol pushed a commit that referenced this pull request Oct 11, 2022

crypto: fix webcrypto generateKey() AES key length validation error

a6e2cb4

PR-URL: #44170 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: #44872

juanarbol pushed a commit that referenced this pull request Oct 11, 2022

crypto: fix webcrypto operation errors to be OperationError

2c938d7

PR-URL: #44171 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: #44872

juanarbol pushed a commit that referenced this pull request Oct 11, 2022

crypto: fix webcrypto EC key namedCurve validation errors

7ad2a26

PR-URL: #44172 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: #44872

juanarbol pushed a commit that referenced this pull request Oct 11, 2022

crypto: fix webcrypto deriveBits validations

7e705d8

PR-URL: #44173 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: #44872

juanarbol pushed a commit that referenced this pull request Oct 11, 2022

crypto: allow zero-length secret KeyObject

e5c9975

PR-URL: #44201 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: #44872

juanarbol pushed a commit that referenced this pull request Oct 11, 2022

crypto: allow zero-length IKM in HKDF and in webcrypto PBKDF2

317cd05

PR-URL: #44201 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Backport-PR-URL: #44872

github-actions bot mentioned this pull request Oct 12, 2022

CI Reliability 2022-10-12 nodejs/reliability#402

Open

19 tasks

panva deleted the backport-lots-to-v16.x branch October 13, 2022 09:12

guangwong pushed a commit to noslate-project/node that referenced this pull request Jan 3, 2023

test: update WPT runner

af7fda4

PR-URL: nodejs/node#43455 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Backport-PR-URL: nodejs/node#44872

guangwong pushed a commit to noslate-project/node that referenced this pull request Jan 3, 2023

test: update WPT runner

63f9a51

PR-URL: nodejs/node#43455 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Backport-PR-URL: nodejs/node#44872

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[v16.x backport] WPT runner update and crypto followups #44872

[v16.x backport] WPT runner update and crypto followups #44872

panva commented Oct 3, 2022 •

edited

Loading

[v16.x backport] WPT runner update and crypto followups #44872

[v16.x backport] WPT runner update and crypto followups #44872

Conversation

panva commented Oct 3, 2022 • edited Loading

panva commented Oct 3, 2022 •

edited

Loading