Skip to content

Update removeChunked to include 204/304#1396

Open
jsmylnycky wants to merge 1 commit intomasterfrom
hotfix/transfer-encoding
Open

Update removeChunked to include 204/304#1396
jsmylnycky wants to merge 1 commit intomasterfrom
hotfix/transfer-encoding

Conversation

@jsmylnycky
Copy link
Copy Markdown
Contributor

No description provided.

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #1396 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1396   +/-   ##
=======================================
  Coverage   92.35%   92.35%           
=======================================
  Files           6        6           
  Lines         314      314           
=======================================
  Hits          290      290           
  Misses         24       24
Impacted Files Coverage Δ
lib/http-proxy/passes/web-outgoing.js 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9bbe486...a2bc4e4. Read the comment docs.

pi0 added a commit to unjs/httpxy that referenced this pull request Mar 25, 2026
Per RFC 9110, 204 and 304 responses must not contain a message body,
so `transfer-encoding` header is meaningless and should be removed.

Upstream: http-party/node-http-proxy#1396
pi0 added a commit to unjs/httpxy that referenced this pull request Mar 25, 2026
Per RFC 9110, 204 and 304 responses must not contain a message body,
so `transfer-encoding` header is meaningless and should be removed.

Upstream: http-party/node-http-proxy#1396
@pi0
Copy link
Copy Markdown

pi0 commented Mar 25, 2026

This issue has been fixed in unjs/httpxy#122.

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.

3 participants