http: unify header treatment by marco-ippolito · Pull Request #46528 · nodejs/node

@marco-ippolito

@nodejs-github-bot added http

Issues or PRs related to the http subsystem.

needs-ci

PRs that need a full CI run.

labels

Feb 6, 2023

climba03003

@marco-ippolito

ShogunPanda

mcollina

ShogunPanda

danielleadams pushed a commit that referenced this pull request

Apr 11, 2023
PR-URL: #46528
Fixes: #46395
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Robert Nagy <ronagy@icloud.com>

codebytere added a commit to electron/electron that referenced this pull request

Apr 17, 2023

codebytere added a commit to electron/electron that referenced this pull request

Apr 17, 2023

codebytere added a commit to electron/electron that referenced this pull request

Apr 18, 2023
* chore: bump node in DEPS to v18.16.0

* build,test: add proper support for IBM i

nodejs/node#46739

* lib: enforce use of trailing commas

nodejs/node#46881

* src: add initial support for single executable applications

nodejs/node#45038

* lib: do not crash using workers with disabled shared array buffers

nodejs/node#41023

* src: remove shadowed variable in OptionsParser::Parse

nodejs/node#46672

* src: allow embedder control of code generation policy

nodejs/node#46368

* src: allow optional Isolate termination in node::Stop()

nodejs/node#46583

* lib: fix BroadcastChannel initialization location

nodejs/node#46864

* chore: fixup patch indices

* chore: sync filenames.json

* fix: add simdutf dep to src/inspector BUILD.gn

- nodejs/node#46471
- nodejs/node#46472

* deps: replace url parser with Ada

nodejs/node#46410

* tls: support automatic DHE

nodejs/node#46978

* fixup! src: add initial support for single executable applications

* http: unify header treatment

nodejs/node#46528

* fix: libc++ buffer overflow in string_view ctor

nodejs/node#46410

* test: include strace openat test

nodejs/node#46150

* fixup! fixup! src: add initial support for single executable applications

---------

Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>

@jk1z jk1z mentioned this pull request

Oct 17, 2023

@lux01 lux01 mentioned this pull request

Nov 27, 2023

This was referenced

Nov 30, 2023

mcollina added a commit to nodejs/undici that referenced this pull request

Jan 25, 2026
Per HTTP spec, header values are byte strings that should be decoded
using isomorphic decode (latin1), not UTF-8. The new interceptor API
(onResponseStart) was incorrectly using UTF-8.

This change:
- Updates parseHeaders and parseRawHeaders to use latin1 encoding
- Removes the content-disposition workaround that was needed when
  headers were inconsistently decoded (see nodejs/node#46528)
- Adds tests to verify latin1 decoding behavior

Fixes #4753

This was referenced

Jan 28, 2026

This was referenced

Jan 30, 2026

This was referenced

Feb 3, 2026

This was referenced

Mar 1, 2026