Make queue-proxy's reverse proxy handler extendable by mbaynton · Pull Request #16097 · knative/serving

@mbaynton

This builds on knative#13133 to make it possible to adjust settings on the
*httputil.ReverseProxy that queue-proxy uses, or to replace it entirely
with any http.Handler, using the out-of-tree extension pattern.

@knative-prow bot added needs-ok-to-test

Indicates a PR that requires an org member to verify it is safe to test.

size/M

Denotes a PR that changes 30-99 lines, ignoring generated files.

labels

Sep 26, 2025

@knative-prow knative-prow bot added ok-to-test

Indicates a non-member PR verified by an org member that is safe to test.

and removed needs-ok-to-test

Indicates a PR that requires an org member to verify it is safe to test.

labels

Sep 28, 2025

dprotaso

@mbaynton

If an integrator customizes the Transport in an Options function, we want to
apply the provided Transport to our default httputil.ReverseProxy
regardless of which Option sets the Transport and regardless of whether
or not other Option functions replace our default http.Handler.

Add some tests for this since the interactions beteween
Options.Transport and Options.ProxyHandler can be subtle.

@knative-prow knative-prow bot added size/L

Denotes a PR that changes 100-499 lines, ignoring generated files.

and removed size/M

Denotes a PR that changes 30-99 lines, ignoring generated files.

labels

Oct 3, 2025

@knative-prow knative-prow bot added the approved

Indicates a PR has been approved by an approver from all required OWNERS files.

label

Oct 6, 2025