rulex123
rulex123
@Ferdudas97 Hello! I have tried this patch out on Spotify's perimeter, which is Envoy based and uses a custom control plane that sends SoTW updates to our Envoy fleet. The...
I would like to start removing support for xDS v2 altogether, do you think it would be possible to coordinate that effort with adding support for xDS incremental? i.e. If...
> > I would like to start removing support for xDS v2 altogether, do you think it would be possible to coordinate that effort with adding support for xDS incremental?...
Ir doesn't look like CI has run for this patch? Could you maybe close and re-open, to see if that triggers a build?
CI fails with some unrelated error (HTTP 404 on downloading some gz file), not sure what to do about that 🤷♀️
OK, I can rework the code to preserve the current behaviour.
@ggreenway Before I change the code, I'd like to make sure we agree on the new API. To recap, the current default behaviour is that all client request headers are...
@ggreenway I reworked the code to preserve backwards compatibility, as you suggested. I _think_ this patch might be ready for a first pass of code review. Could you take a...