Skip to content

Add plain HTTP egress forwarding - #1240

Open
Weidong Cui (wdcui) wants to merge 1 commit into
wdcui/ulitebox/egress-proxy-connectfrom
wdcui/ulitebox/egress-proxy-http
Open

Add plain HTTP egress forwarding#1240
Weidong Cui (wdcui) wants to merge 1 commit into
wdcui/ulitebox/egress-proxy-connectfrom
wdcui/ulitebox/egress-proxy-http

Conversation

@wdcui

Copy link
Copy Markdown
Member

This PR adds plain HTTP/1 forwarding to the standalone proxy, including absolute-to-origin request rewriting, canonical Host forwarding, hop-by-hop and framing validation, per-request upstream connections, bounded body streaming, and hermetic loopback coverage.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 239ec5f7-870a-4259-bcae-4ca85fb913a0
@github-actions

Copy link
Copy Markdown

ℹ️ Note: This semver check was run against the wdcui/ulitebox/egress-proxy-connect branch, not main or ulitebox.

🤖 SemverChecks 🤖 ⚠️ Potential breaking API changes detected ⚠️

Click for details
--- failure enum_no_repr_variant_discriminant_changed: enum variant had its discriminant change value ---

Description:
The enum's variant had its discriminant value change. This breaks downstream code that used its value via a numeric cast like `as isize`.
        ref: https://doc.rust-lang.org/reference/items/enumerations.html#assigning-discriminant-values
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.50.0/src/lints/enum_no_repr_variant_discriminant_changed.ron

Failed in:
  variant FramingError::DuplicateHost 0 -> 6 in /home/runner/work/litebox/litebox/litebox_egress_proxy/src/headers.rs:52
  variant FramingError::BodyOnConnect 1 -> 7 in /home/runner/work/litebox/litebox/litebox_egress_proxy/src/headers.rs:55

--- failure enum_variant_added: enum variant added on exhaustive enum ---

Description:
A publicly-visible enum without #[non_exhaustive] has a new variant.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#enum-variant-new
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.50.0/src/lints/enum_variant_added.ron

Failed in:
  variant FramingError:ConflictingLength in /home/runner/work/litebox/litebox/litebox_egress_proxy/src/headers.rs:34
  variant FramingError:ConflictingContentLength in /home/runner/work/litebox/litebox/litebox_egress_proxy/src/headers.rs:37
  variant FramingError:InvalidContentLength in /home/runner/work/litebox/litebox/litebox_egress_proxy/src/headers.rs:40
  variant FramingError:UnsupportedTransferCoding in /home/runner/work/litebox/litebox/litebox_egress_proxy/src/headers.rs:43
  variant FramingError:TransferEncodingOnHttp10 in /home/runner/work/litebox/litebox/litebox_egress_proxy/src/headers.rs:46
  variant FramingError:NonAsciiHeaderValue in /home/runner/work/litebox/litebox/litebox_egress_proxy/src/headers.rs:49

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