Steven Kreitzer
Steven Kreitzer
> rethinking this one, shouldnt this be enabled by default ? > > what do HAProxy, NGINX and others do ? They pass it directly to the backend, this is...
@arkodg I just made this `removeAcceptEncodingHeader` default.
@arkodg Can you please review?
> since this's a behavior change, please add it in the release-notes. @zirain Done :)
> have 2 open questions > > * why does this PR impact chooseFirst ? which seems unrelated to removing the header > * should this be enabled by default...
Unfortunately, there was a `release-notes/current.yaml` conflict due to https://github.com/envoyproxy/gateway/commit/92d2a1a5a99deb3842b1cf888b2639d2a8c6cd7d#diff-8261eaec1f1079fe46bee37a988c04a7876b25a7c93fdc67db2f918ee5e406fb. Rebased and updated.
Not sure if this helps, but this is how we do it: https://github.com/buroa/k8s-gitops/blob/main/kubernetes/apps/networking/envoy-gateway/config/envoy.yaml#L57