Skip to content

[24.x] deps: update nghttp2 to 1.69.0 - #63164

Closed
mcollina wants to merge 2 commits into
nodejs:v24.x-stagingfrom
mcollina:nghttp2-main-updates-v24.x-staging
Closed

[24.x] deps: update nghttp2 to 1.69.0#63164
mcollina wants to merge 2 commits into
nodejs:v24.x-stagingfrom
mcollina:nghttp2-main-updates-v24.x-staging

Conversation

@mcollina

@mcollinamcollina commented May 7, 2026

Copy link
Copy Markdown
Member

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/http2
  • @nodejs/net
  • @nodejs/security-wg

@nodejs-github-botnodejs-github-bot added lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. v24.x Issues that can be reproduced on v24.x or PRs targeting the v24.x-staging branch. labels May 7, 2026
@codecov

codecovBot commented May 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 87.50000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.91%. Comparing base (b881658) to head (2756a4d).
⚠️ Report is 66 commits behind head on v24.x-staging.

Files with missing linesPatch %Lines
src/node_http2.cc87.50%0 Missing and 2 partials ⚠️
Additional details and impacted files
@@ Coverage Diff @@## v24.x-staging #63164 +/- ##
=================================================
- Coverage 89.92% 89.91% -0.01% 
=================================================
Files 686 686 Lines 208389 208993 +604 Branches 40077 40184 +107 =================================================
+ Hits 187387 187924 +537 - Misses 13238 13293 +55 - Partials 7764 7776 +12 
Files with missing linesCoverage Δ
src/node_http2.h91.61% <ø> (ø)
src/node_http2.cc81.85% <87.50%> (-0.01%)⬇️

... and 53 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mcollinamcollina changed the title Nghttp2 main updates v24.x staging[24.x] deps: update nghttp2 to 1.69.0May 7, 2026
@mcollinamcollina added the request-ci Add this label to start a Jenkins CI on a PR. label May 7, 2026
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 7, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@aduh95

Copy link
Copy Markdown
Contributor

@nodejs/lts how confortable are we to land this on LTS release line given the test changes?

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@mcollina

Copy link
Copy Markdown
MemberAuthor

We need to backport this update due to GHSA-6933-cjhr-5qg6. I assume we don't want to float a patch on top of nghttp2, which also would not please security scanners.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@aduh95aduh95 added the request-ci Add this label to start a Jenkins CI on a PR. label May 26, 2026
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 26, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

nodejs-github-botand others added 2 commits May 27, 2026 17:30
PR-URL: nodejs#62891
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Stephen Belanger <admin@stephenbelanger.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
This is a set of src & tests fixes for nghttp2 due to changes in
v1.67.0+ which require a selection of changes to how we handle
low-level protocol errors when using the latest versions of nghttp2,
changing both some src error handling and updating some tests to match.
Signed-off-by: Tim Perry <pimterry@gmail.com>
PR-URL: nodejs#62891
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Stephen Belanger <admin@stephenbelanger.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
@aduh95
aduh95force-pushed the nghttp2-main-updates-v24.x-staging branch from 3fb51d5 to 2756a4dCompareMay 27, 2026 15:31
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

aduh95 pushed a commit that referenced this pull request Jun 16, 2026
PR-URL: #62891
Backport-PR-URL: #63164
Reviewed-By: Tim Perry <pimterry@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
aduh95 pushed a commit that referenced this pull request Jun 16, 2026
This is a set of src & tests fixes for nghttp2 due to changes in
v1.67.0+ which require a selection of changes to how we handle
low-level protocol errors when using the latest versions of nghttp2,
changing both some src error handling and updating some tests to match.
Signed-off-by: Tim Perry <pimterry@gmail.com>
PR-URL: #62891
Backport-PR-URL: #63164
Reviewed-By: Tim Perry <pimterry@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
@aduh95

Copy link
Copy Markdown
Contributor

Landed in dd627ce...a8a0d12

@aduh95aduh95 closed this Jun 18, 2026
panva pushed a commit to panva/node that referenced this pull request Jun 19, 2026
PR-URL: nodejs#62891
Backport-PR-URL: nodejs#63164
Reviewed-By: Tim Perry <pimterry@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
panva pushed a commit to panva/node that referenced this pull request Jun 19, 2026
This is a set of src & tests fixes for nghttp2 due to changes in
v1.67.0+ which require a selection of changes to how we handle
low-level protocol errors when using the latest versions of nghttp2,
changing both some src error handling and updating some tests to match.
Signed-off-by: Tim Perry <pimterry@gmail.com>
PR-URL: nodejs#62891
Backport-PR-URL: nodejs#63164
Reviewed-By: Tim Perry <pimterry@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Refs: https://hackerone.com/reports/3658225
CVE-ID: CVE-2026-48937
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lib / srcIssues and PRs related to general changes in the lib or src directory.needs-ciPRs that need a full CI run.v24.xIssues that can be reproduced on v24.x or PRs targeting the v24.x-staging branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@mcollina@nodejs-github-bot@aduh95@pimterry@trivikr@RafaelGSS