deps: update undici to 7.0.0

[nodejs-github-bot]

This is an automated update of undici to 7.0.0.

[nodejs-github-bot]

Review requested:

• @nodejs/security-wg

[aduh95]

What's the semverness of this @nodejs/undici ?

[github-actions]

The notable-change PRs with changes that should be highlighted in changelogs. label has been added by @mcollina.

Please suggest a text for the release notes if you'd like to include a more detailed summary, then proceed to update the PR description with the text or a link to the notable change suggested text comment. Otherwise, the commit will be placed in the Other Notable Changes section.

[mcollina]

I would consider it semver-major, or at least something to not backport on lts lines.

The major relevant change are

Stricter fetch() spec compliance

Back when we first considered fetch() to Node.js, we needed to provide a migration path for all the users of the various polyfills that where on the NPM registry. Therefore, we added support for third party Blob, FormData, and AbortController. This support was tested but not really documented. However, this causes quite a few maintenance issues as those polyfills do not follow the spec.

In Undici v7, we decided to remove support for those entirely in our fetch() implementation. Therefore, only the provided classes will work with it.

Updates to llhttp v9

The new version of the HTTP parser switches to an always strict parsing logic.

If you only connect to HTTP specification compliant servers, this change will not impact you. If this is an important issue for you, we recommend checking nodejs/undici#2678 out, and contribute back.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.00%. Comparing base (4efb7ae) to head (f119991).
Report is 14 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #56070   +/-   ##
=======================================
  Coverage   87.99%   88.00%           
=======================================
  Files         656      656           
  Lines      188988   189000   +12     
  Branches    35988    35990    +2     
=======================================
+ Hits       166302   166326   +24     
+ Misses      15848    15836   -12     
  Partials     6838     6838           

see 23 files with indirect coverage changes

[nodejs-github-bot]

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

[targos]

I see it updates llhttp to v9. Is it a notable change?

[mcollina]

It is!

[mcollina]

After this lands, we could add an experimental WebSocketStream implementation to Node.js as well.

[RafaelGSS]

RSLGTM

[mcollina]

lgtm

[mcollina]

lgtm

[nodejs-github-bot]

Commit Queue failed

- Loading data for nodejs/node/pull/56070
✔  Done loading data for nodejs/node/pull/56070
----------------------------------- PR info ------------------------------------
Title      deps: update undici to 7.0.0 (#56070)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     nodejs-github-bot:actions/tools-update-undici -> nodejs:main
Labels     semver-major, notable-change, author ready, needs-ci, dependencies, commit-queue-squash
Commits    2
 - deps: update undici to 7.0.0
 - deps: update undici to 7.0.0
Committers 1
 - Node.js GitHub Bot <github-bot@iojs.org>
PR-URL: https://github.com/nodejs/node/pull/56070
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Matthew Aitken <maitken033380023@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/56070
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Matthew Aitken <maitken033380023@gmail.com>
--------------------------------------------------------------------------------
   ℹ  This PR was created on Fri, 29 Nov 2024 09:12:39 GMT
   ✔  Approvals: 4
   ✔  - Michaël Zasso (@targos) (TSC): https://github.com/nodejs/node/pull/56070#pullrequestreview-2469649863
   ✔  - Rafael Gonzaga (@RafaelGSS) (TSC): https://github.com/nodejs/node/pull/56070#pullrequestreview-2469755387
   ✔  - Matteo Collina (@mcollina) (TSC): https://github.com/nodejs/node/pull/56070#pullrequestreview-2471340963
   ✔  - Matthew Aitken (@KhafraDev): https://github.com/nodejs/node/pull/56070#pullrequestreview-2470117914
   ✔  Last GitHub CI successful
   ℹ  Last Full PR CI on 2024-11-29T10:41:07Z: https://ci.nodejs.org/job/node-test-pull-request/63771/
   ⚠  Commits were pushed after the last Full PR CI run:
   ⚠  - deps: update undici to 7.0.0
- Querying data for job/node-test-pull-request/63771/
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu

https://github.com/nodejs/node/actions/runs/12107971135

[nodejs-github-bot]

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

[mhdawson]

Rubber stamp LGTM

[nodejs-github-bot]

Landed in 47b80c2