Skip to content

fix(proxy): preserve streamed HTTP request trailers - #64

Merged
cppla merged 1 commit into
mainfrom
codex/http-forward-request-trailers
Oct 3, 2026
Merged

cppla merged 1 commit into
mainfrom
codex/http-forward-request-trailers

Conversation

@cppla

@cppla cppla commented Oct 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • Bridge safe declared request trailers from the inbound body at successful EOF, preserving streamed HTTP forward-proxy uploads without buffering.
  • Filter forbidden trailer fields, proxy credentials and initial/late Connection nominations; preserve ordinary origin Authorization and body Read/Close errors.
  • Add real TCP streaming, filtering and visitor-cancellation regressions plus ownership/framing/EOF unit coverage.

Validation

  • Original controlled baseline: direct upload 200, proxied upload 422, identical complete 65,536-byte body; only safe trailers lost.
  • Fixed real-wire controls: direct/proxy 200 with complete body/digest and first-body-before-EOF witness.
  • Full make check and make race; Go 1.27.1 focused race x10; Go 1.25.13 focused race x3; offline Linux/arm64 Docker Go 1.25.13 race x3.

Boundary

A nomination received only at EOF filters values but cannot retract a declaration already streamed upstream. No dependency, protocol, default, version, release/tag or registry changes. This is upload compatibility evidence, not traffic-identification evidence.

Copilot AI balanced review requested due to automatic review settings October 3, 2026 08:59

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@cppla
cppla merged commit 1b37397 into main Oct 3, 2026
14 checks passed
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.

2 participants