Skip to content

deps: update undici to 5.16.0#46213

Merged
nodejs-github-bot merged 3 commits intomainfrom
actions/tools-update-undici
Jan 27, 2023
Merged

deps: update undici to 5.16.0#46213
nodejs-github-bot merged 3 commits intomainfrom
actions/tools-update-undici

Conversation

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot nodejs-github-bot commented Jan 14, 2023

This is an automated update of undici to 5.15.0.

@nodejs-github-bot nodejs-github-bot added dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run. labels Jan 14, 2023
@debadree25
Copy link
Member

debadree25 commented Jan 15, 2023

Could we rerun this workflow, in reference to #46217

@lpinca lpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Jan 19, 2023
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 19, 2023
@nodejs-github-bot
Copy link
Collaborator Author

nodejs-github-bot commented Jan 19, 2023

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

@richardlau richardlau added the commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. label Jan 19, 2023
@richardlau
Copy link
Member

richardlau commented Jan 19, 2023

This has relevant looking test failures for the withoutssl build. @nodejs/undici

@ronag
Copy link
Member

ronag commented Jan 19, 2023

This has relevant looking test failures for the withoutssl build. @nodejs/undici

Where do I see those?

@richardlau
Copy link
Member

richardlau commented Jan 19, 2023

https://ci.nodejs.org/job/node-test-commit-linux-containered/nodes=ubuntu1804_sharedlibs_withoutssl_x64/35531/consoleFull

Here's the first few failures:

09:17:10 python3 ./configure --verbose  --without-ssl
...
09:20:58 not ok 949 parallel/test-fetch
09:20:58   ---
09:20:58   duration_ms: 0.192
09:20:58   severity: fail
09:20:58   exitcode: 1
09:20:58   stack: |-
09:20:58     node:internal/errors:490
09:20:58         ErrorCaptureStackTrace(err);
09:20:58         ^
09:20:58     
09:20:58     Error [ERR_NO_CRYPTO]: Node.js is not compiled with OpenSSL crypto support
09:20:58         at new NodeError (node:internal/errors:399:5)
09:20:58         at assertCrypto (node:internal/util:171:11)
09:20:58         at node:crypto:36:1
09:20:58         at BuiltinModule.compileForInternalLoader (node:internal/bootstrap/loaders:333:7)
09:20:58         at requireBuiltin (node:internal/bootstrap/loaders:364:14)
09:20:58         at requireWithFallbackInDeps (node:internal/bootstrap/loaders:373:10)
09:20:58         at lib/websocket/connection.js (node:internal/deps/undici/undici:14216:39)
09:20:58         at __require (node:internal/deps/undici/undici:4:50)
09:20:58         at lib/websocket/websocket.js (node:internal/deps/undici/undici:14394:44)
09:20:58         at __require (node:internal/deps/undici/undici:4:50) {
09:20:58       code: 'ERR_NO_CRYPTO'
09:20:58     }
09:20:58     
09:20:58     Node.js v20.0.0-pre
09:20:58   ...
...
09:22:04 not ok 1472 parallel/test-http-response-setheaders
09:22:04   ---
09:22:04   duration_ms: 0.176
09:22:04   severity: fail
09:22:04   exitcode: 1
09:22:04   stack: |-
09:22:04     node:internal/util:171
09:22:04         throw new ERR_NO_CRYPTO();
09:22:04         ^
09:22:04     
09:22:04     Error [ERR_NO_CRYPTO]: Node.js is not compiled with OpenSSL crypto support
09:22:04         at new NodeError (node:internal/errors:399:5)
09:22:04         at assertCrypto (node:internal/util:171:11)
09:22:04         at node:crypto:36:1
09:22:04         at BuiltinModule.compileForInternalLoader (node:internal/bootstrap/loaders:333:7)
09:22:04         at requireBuiltin (node:internal/bootstrap/loaders:364:14)
09:22:04         at requireWithFallbackInDeps (node:internal/bootstrap/loaders:373:10)
09:22:04         at lib/websocket/connection.js (node:internal/deps/undici/undici:14216:39)
09:22:04         at __require (node:internal/deps/undici/undici:4:50)
09:22:04         at lib/websocket/websocket.js (node:internal/deps/undici/undici:14394:44)
09:22:04         at __require (node:internal/deps/undici/undici:4:50) {
09:22:04       code: 'ERR_NO_CRYPTO'
09:22:04     }
09:22:04     
09:22:04     Node.js v20.0.0-pre
09:22:04   ...
...
09:23:55 not ok 3122 parallel/test-wasm-web-api
09:23:55   ---
09:23:55   duration_ms: 0.219
09:23:55   severity: fail
09:23:55   exitcode: 1
09:23:55   stack: |-
09:23:55     node:internal/process/promises:289
09:23:55                 triggerUncaughtException(err, true /* fromPromise */);
09:23:55                 ^
09:23:55     
09:23:55     AssertionError [ERR_ASSERTION]: Expected values to be strictly deep-equal:
09:23:55     + actual - expected
09:23:55     
09:23:55       Comparison {
09:23:55     +   code: 'ERR_NO_CRYPTO',
09:23:55     +   message: 'Node.js is not compiled with OpenSSL crypto support',
09:23:55     +   name: 'Error'
09:23:55     -   code: 'ERR_INVALID_ARG_TYPE',
09:23:55     -   message: /^The "source" argument .*$/,
09:23:55     -   name: 'TypeError'
09:23:55       }
09:23:55         at async Promise.all (index 1)
09:23:55         at async /home/iojs/build/workspace/node-test-commit-linux-containered/test/parallel/test-wasm-web-api.js:73:5 {
09:23:55       generatedMessage: true,
09:23:55       code: 'ERR_ASSERTION',
09:23:55       actual: Error [ERR_NO_CRYPTO]: Node.js is not compiled with OpenSSL crypto support
09:23:55           at new NodeError (node:internal/errors:399:5)
09:23:55           at assertCrypto (node:internal/util:171:11)
09:23:55           at node:crypto:36:1
09:23:55           at BuiltinModule.compileForInternalLoader (node:internal/bootstrap/loaders:333:7)
09:23:55           at requireBuiltin (node:internal/bootstrap/loaders:364:14)
09:23:55           at requireWithFallbackInDeps (node:internal/bootstrap/loaders:373:10)
09:23:55           at lib/websocket/connection.js (node:internal/deps/undici/undici:14216:39)
09:23:55           at __require (node:internal/deps/undici/undici:4:50)
09:23:55           at lib/websocket/websocket.js (node:internal/deps/undici/undici:14394:44)
09:23:55           at __require (node:internal/deps/undici/undici:4:50) {
09:23:55         code: 'ERR_NO_CRYPTO'
09:23:55       },
09:23:55       expected: {
09:23:55         name: 'TypeError',
09:23:55         code: 'ERR_INVALID_ARG_TYPE',
09:23:55         message: /^The "source" argument .*$/
09:23:55       },
09:23:55       operator: 'rejects'
09:23:55     }
09:23:55     
09:23:55     Node.js v20.0.0-pre
09:23:55   ...
...
09:26:00 Failed tests:
09:26:00 out/Release/node /home/iojs/build/workspace/node-test-commit-linux-containered/test/parallel/test-fetch.mjs
09:26:00 out/Release/node /home/iojs/build/workspace/node-test-commit-linux-containered/test/parallel/test-http-response-setheaders.js
09:26:00 out/Release/node /home/iojs/build/workspace/node-test-commit-linux-containered/test/parallel/test-wasm-web-api.js
09:26:00 out/Release/node /home/iojs/build/workspace/node-test-commit-linux-containered/test/wpt/test-structured-clone.js
09:26:00 out/Release/node /home/iojs/build/workspace/node-test-commit-linux-containered/test/wpt/test-url.js
09:26:00 out/Release/node /home/iojs/build/workspace/node-test-commit-linux-containered/test/wpt/test-wasm-webapi.js

@mcollina
Copy link
Member

mcollina commented Jan 19, 2023

@KhafraDev I think we should avoid loading WebSocket if there is no crypto, and definitely do not crash.

This was referenced Jan 20, 2023
Open
Open
Open
@richardlau richardlau removed the commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. label Jan 22, 2023
@github-actions github-actions bot mentioned this pull request Jan 23, 2023
Open
13 tasks
@aduh95 aduh95 force-pushed the actions/tools-update-undici branch from 323d837 to 16da87b Compare January 23, 2023 08:55
@aduh95 aduh95 added the request-ci Add this label to start a Jenkins CI on a PR. label Jan 23, 2023
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 23, 2023
@nodejs-github-bot
Copy link
Collaborator Author

nodejs-github-bot commented Jan 23, 2023

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

@richardlau richardlau added the commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. label Jan 23, 2023
This was referenced Jan 24, 2023
Open
Open
@mcollina mcollina changed the title deps: update undici to 5.15.0 deps: update undici to 5.16.0 Jan 26, 2023
@mcollina mcollina added the request-ci Add this label to start a Jenkins CI on a PR. label Jan 26, 2023
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 26, 2023
@richardlau richardlau added the request-ci Add this label to start a Jenkins CI on a PR. label Jan 27, 2023
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 27, 2023
@nodejs-github-bot
Copy link
Collaborator Author

nodejs-github-bot commented Jan 27, 2023

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

mcollina
mcollina approved these changes Jan 27, 2023
View reviewed changes
Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@mcollina mcollina added commit-queue Add this label to land a pull request using GitHub Actions. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. and removed commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. labels Jan 27, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 27, 2023
@nodejs-github-bot nodejs-github-bot merged commit 19623e3 into main Jan 27, 2023
@nodejs-github-bot nodejs-github-bot deleted the actions/tools-update-undici branch January 27, 2023 13:50
@nodejs-github-bot
Copy link
Collaborator Author

nodejs-github-bot commented Jan 27, 2023

Landed in 19623e3

@richardlau
Copy link
Member

richardlau commented Jan 27, 2023

@mcollina FYI Unfortunately squashing the commits using the commit queue has given the squashed commit the title from the first commit: 19623e3
"deps: update undici to 5.15.0" which is going to be confusing in the changelogs.

The commit queue works this way because the more common squash flow is that the follow up commits being squashed are fix ups.

ruyadorno pushed a commit that referenced this pull request Feb 1, 2023
@nodejs-github-bot @ruyadorno
deps: update undici to 5.15.0
9c2f3ce 
PR-URL: #46213
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
@ruyadorno ruyadorno mentioned this pull request Feb 1, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mcollina mcollina mcollina approved these changes

@anonrig anonrig anonrig approved these changes

@richardlau richardlau richardlau approved these changes

Assignees

No one assigned

Labels

commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@nodejs-github-bot @debadree25 @richardlau @ronag @mcollina @anonrig @lpinca @aduh95