htuch comments

Results 96 comments of


                                            htuch

Updating LbEndpoint metadata causes connection churn

Is this `LbEndpoint` metadata or `LocalityLbEndpoints`?

Updating LbEndpoint metadata causes connection churn

@cpakulski @adisuissa

Introduce QuicListenerFilter::onFirstPacketReceived

/wait

WASM Calling `proxy_send_local_response` twice will stuck remote http client(e.g. curl) forever until timeout or interrupted

@mpwarres

OAuth2: add retry policy to the OAuth2 filter

I don't think retry policy is a property of a URI but of a service. So you could potentially add it to the recently added `envoy.config.core.v3.HttpService`. But in this case...

OAuth2: add retry policy to the OAuth2 filter

I agree on the history. Going forward, can we deprecate the existing URI field and replace with Service, adding this there?

OAuth2: add retry policy to the OAuth2 filter

I'm fine with retry policy. where it is in this PR as long as we don't put it in `HttpUri`.

uri_template: Allow uri_template variables in literal expression

@yanavlasov friendly ping.

Configuring an ApiConfigSource on a dynamic cluster

I believe this is a limit we introduced to simplify some of the bootstrap dependency issues across cluster initialization and warming. @yanavlasov probably has some of the latest context on...

Configuring an ApiConfigSource on a dynamic cluster

Re: https://github.com/envoyproxy/envoy/issues/12954#issuecomment-692207481, the top-level `cluster_names` only applies to REST (see comment on the field). It should prefer the gRPC cluster name if you skip configuring top-level `cluster_names`; I'd say this...