Arko Dasgupta
Arko Dasgupta
@modatwork I believe one already exists on 0.0.0.0:9001 https://gateway.envoyproxy.io/v1.0.1/install/install-helm/#envoyproxy https://github.com/envoyproxy/gateway/pull/1568 we probably need to surface this info in a better way in docs
sure +1 for adding this in ClientTrafficPolicy
https://github.com/envoyproxy/gateway/issues/3163 should fix this, BTP support for TCPRoute was recently added ( available.on v0.0.0-latest)
@bhamon rethinking, for your use case (TCP proxying) you dont even need a CTP with `enableProxyProtocol` set, since its TCP, it should proxy the proxy protocol header over TCP ....
thanks for running the tests @bhamon ! , you're right reg TLS, #3163 is blocking it
thanks for taking the PR forward @aoledk ! will try to review this one soon before another round of merges making this refractor hard the keep alive issue is being...
hey @aoledk there are some CI failures, running `make generate` and fixing the new format for the xds test should fix it
met @vixns at Kubecon EU who brought this up in person :) Lets gather some more info on why ingress-nginx and contour prioritize regex over prefix @AliceProxy curious to know...
finally took a look at the config in the PR where `PathPrefix` ``` type: PathPrefix value: / ``` is used as a fallback This implies that if EG wants to...
hey @vixns added some suggestions, mainly around code style for this function