Skip to content

fix(res.append): preserve empty header values#7332

Open
lin-hongkuan wants to merge 1 commit into
expressjs:masterfrom
lin-hongkuan:codex/express-append-empty-header
Open

fix(res.append): preserve empty header values#7332
lin-hongkuan wants to merge 1 commit into
expressjs:masterfrom
lin-hongkuan:codex/express-append-empty-header

Conversation

@lin-hongkuan

@lin-hongkuan lin-hongkuan commented Jun 24, 2026

Copy link
Copy Markdown

This makes res.append() preserve an existing empty-string header value instead of treating it as absent.

The regression test appends to an empty Warning header and verifies the empty value remains part of the emitted header list.

Validation:

  • npx mocha --require test/support/env --check-leaks test/res.append.js
  • npm run lint
  • git diff --check

AI-assisted: Codex helped prepare this focused change; I reviewed the diff and ran the validation above before submitting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant