chore(deps): Bump undici from 7.29.0 to 8.10.0 - #496
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF Scorecard
Scanned Files
|
|
@dependabot rebase |
Bumps [undici](https://github.com/nodejs/undici) from 7.29.0 to 8.10.0. - [Release notes](https://github.com/nodejs/undici/releases) - [Commits](nodejs/undici@v7.29.0...v8.10.0) --- updated-dependencies: - dependency-name: undici dependency-version: 8.10.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
c40cedc to
95815cb
Compare
…osing it #496 still failed after the Node 20 drop and a rebase onto [22.x, 24.x] - on both legs. Every integration file that does real HTTP through our dispatcher either timed out or failed with "EFATAL: fetch failed". The cause is not the Node version. We build an Agent from the standalone undici package and node-telegram-bot-api hands it to Node's BUILT-IN fetch, which comes from Node's own bundled undici. undici v8 removed the legacy dispatcher handler wrappers that the bundled copy still drives a dispatcher through, so the dispatcher is accepted - dispatch() is reached, which is why the unit test kept passing - but no response ever comes back. - undici stays at ^7; .github/dependabot.yml ignores its majors so the PR does not return weekly. Minor/patch still flow, so security fixes are not blocked. - ADR 0011 records the finding, the alternatives, and what has to be checked before the ignore is lifted (process.versions.undici on the minimum Node). - ADR 0010 is corrected: it was framed as though undici 8 were being adopted. The Node 20 drop stands on its own - node-red@5 requires >=22.9 and Node 20 is EOL - and now says so. - CHANGELOG and MIGRATION.md follow the same correction. No release exists for 19.0.0 yet, so this amends that entry in place. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Looks like undici is no longer being updated by Dependabot, so this is no longer needed. |
|
Closing unmerged — undici 8 is not usable by this package yet, and it is not a Node-version problem. After
Every failure is a request that goes out and never comes back. Why. Two undici majors cannot be mixed across the built-in-fetch boundary. This will become mergeable once a Node release bundles undici 8;
Full reasoning: ADR 0011. |
|
Follow-up: the diagnosis above has now been verified directly, not just inferred. When this was closed, the reasoning rested on the CI failure pattern plus undici's release notes. A machine with Node 24.19 was available afterwards, so the claim could be tested. It holds — and the mechanism is more precise than originally described. Against a local HTTP server, with
The full suite agrees: 14 integration subtests pass on One number worth recording: Node 24.19.0 still bundles undici 7.29.0 — the newest line in our CI matrix. The wait for a bundled undici 8 is real, so the Dependabot ignore rule is not about to become stale. Nothing changed in |
Bumps undici from 7.29.0 to 8.10.0.
Release notes
Sourced from undici's releases.
... (truncated)
Commits
c8d80e6Bumped v8.10.0 (#5644)66923b4fix: preserve DNS origin hostname on sockets (#5577)3926499fix: retry refused HTTP/2 streams (#5598)73d6e9efix(h2): detach upgrade close handler after GOAWAY (#5641)b111adbfix(mock): emit request body lifecycle hooks (#5367)ae4a3e3build(deps): bump actions/setup-node from 6.4.0 to 7.0.0 (#5636)ec3fbf1build(deps): bump github/codeql-action/init from 4.36.2 to 4.37.3 (#5634)2151720build(deps): bump ossf/scorecard-action from 2.4.3 to 2.4.4 (#5633)b96a116fix(interceptors): allow interceptors without opts.origin (#5628)a18ef2dfix(mock): non-string path matchers under ignoreTrailingSlash, and DataView r...