quic: convert incoming :status header to number

[hallss93]

Fixes: #63557

Summary

When receiving HTTP/3 headers via QUIC, headers[':status'] was exposed as a
string. HTTP/2 converts the incoming :status pseudo-header to a number in
toHeaderObject() (lib/internal/http2/util.js). This mismatch makes it
awkward to port HTTP/2 code to HTTP/3.

Align HTTP/3 with HTTP/2 by converting :status to a number in
parseHeaderPairs() when building the header object from C++ name/value
pairs. This applies to initial response headers, informational (1xx)
headers (oninfo), and stream.headers.

Outgoing headers are unchanged: callers may still pass ':status': '200' or
':status': 200 when calling sendHeaders() / sendInformationalHeaders().

Test plan

• Update existing QUIC H3 tests to expect numeric :status on receive

• Add test/parallel/test-quic-h3-status-code-type.mjs covering 200, 204, and 404

• Run (with QUIC enabled):

  node --experimental-quic --experimental-stream-iter --no-warnings \
    test/parallel/test-quic-h3-status-code-type.mjs \
    test/parallel/test-quic-h3-header-validation.mjs \
    test/parallel/test-quic-h3-informational-headers.mjs \
    test/parallel/test-quic-h3-request-response.mjs

[nodejs-github-bot]

Review requested:

• @nodejs/quic

[pimterry]

@hallss93 Looks like the commit message here is failing the linting as it's not signed off, you'll need to update that with git commit --amend -s and then force push.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.32%. Comparing base (92f48f4) to head (f442353).
⚠️ Report is 22 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #63589      +/-   ##
==========================================
+ Coverage   90.31%   90.32%   +0.01%     
==========================================
  Files         730      730              
  Lines      234653   234658       +5     
  Branches    43952    43951       -1     
==========================================
+ Hits       211926   211959      +33     
+ Misses      14474    14430      -44     
- Partials     8253     8269      +16     

Files with missing lines	Coverage Δ
lib/internal/quic/quic.js	100.00% <100.00%> (ø)

... and 32 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.

[hallss93]

@hallss93 Looks like the commit message here is failing the linting as it's not signed off, you'll need to update that with git commit --amend -s and then force push.

fixed

[nodejs-github-bot]

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

[nodejs-github-bot]

Commit Queue failed

- Loading data for nodejs/node/pull/63589
✔  Done loading data for nodejs/node/pull/63589
----------------------------------- PR info ------------------------------------
Title      quic: convert incoming :status header to number (#63589)
Author     Hallison Pereira Melo <hallss93@hotmail.com> (@hallss93, first-time contributor)
Branch     hallss93:quic-h3-status-number -> nodejs:main
Labels     needs-ci, quic, commit-queue-squash
Commits    2
 - quic: convert incoming :status header to number
 - Merge branch 'nodejs:main' into quic-h3-status-number
Committers 2
 - Hallison Melo <hallss93@hotmail.com>
 - GitHub <noreply@github.com>
PR-URL: https://github.com/nodejs/node/pull/63589
Fixes: https://github.com/nodejs/node/issues/63557
Reviewed-By: Stephen Belanger <admin@stephenbelanger.com>
Reviewed-By: Tim Perry <pimterry@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/63589
Fixes: https://github.com/nodejs/node/issues/63557
Reviewed-By: Stephen Belanger <admin@stephenbelanger.com>
Reviewed-By: Tim Perry <pimterry@gmail.com>
--------------------------------------------------------------------------------
   ℹ  This PR was created on Tue, 26 May 2026 18:45:32 GMT
   ✔  Approvals: 2
   ✔  - Stephen Belanger (@Qard): https://github.com/nodejs/node/pull/63589#pullrequestreview-4378129270
   ✔  - Tim Perry (@pimterry): https://github.com/nodejs/node/pull/63589#pullrequestreview-4449032505
   ✔  Last GitHub CI successful
   ℹ  Last Full PR CI on 2026-06-08T13:56:50Z: https://ci.nodejs.org/job/node-test-pull-request/73976/
- Querying data for job/node-test-pull-request/73976/
✔  Build data downloaded
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  No git cherry-pick in progress
   ✔  No git am in progress
   ✔  No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
 * branch                  main       -> FETCH_HEAD
✔  origin/main is now up-to-date
- Downloading patch for 63589
From https://github.com/nodejs/node
 * branch                  refs/pull/63589/merge -> FETCH_HEAD
✔  Fetched commits as 45c7071b4a0d..04c9e28facb5
--------------------------------------------------------------------------------
Auto-merging doc/api/quic.md
Auto-merging lib/internal/quic/quic.js
error: commit 04c9e28facb599da6a23142450bbf8c85e32bcd0 is a merge but no -m option was given.
fatal: cherry-pick failed
[main dad2bd067b] quic: convert incoming :status header to number
 Author: Hallison Melo <hallss93@hotmail.com>
 Date: Tue May 26 15:40:36 2026 -0300
 22 files changed, 117 insertions(+), 44 deletions(-)
 create mode 100644 test/parallel/test-quic-h3-status-code-type.mjs
   ✘  Failed to apply patches

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

[trivikr]

@hallss93 Can you please rebase onto main and remove the merge commit?

Instructions: https://github.com/nodejs/node/blob/main/doc/contributing/pull-requests.md#step-5-rebase

[bjohansebas]

LGTM!