Add configurable listener ALPN protocols to enable downstream http2

uswitch / yggdrasil

Envoy Control Plane for Kubernetes Multi-cluster Ingress

Apache License 2.0

192 stars 17 forks source link

Open Aluxima opened 1 year ago

Aluxima commented 1 year ago

No breaking change, listener offers no ALPN protocol by default.

Aluxima commented 1 year ago

Marking this PR as draft as envoy has issues with http2 connection coalescing, leading to 404s. See https://github.com/envoyproxy/envoy/issues/6767

We could add a lua script to respond with a 421 when a connection is being used for the wrong host, like Contour did: https://github.com/projectcontour/contour/pull/2483

DewaldV commented 1 year ago

I hope that issued gets resolved, I'd love to get Downstream HTTP/2 enabled as well.

I'll give the Lua solution a peek as well though I can't say I'm Lua proficient :joy: