envoyproxy/gateway

Fix conformance for GatewayHTTPSListenerDetectMisdirectedRequests

Offen

#5.879 geöffnet am 01.05.2025

Auf GitHub ansehen
 (14 Kommentare) (2 Reaktionen) (1 zugewiesene Person)Go (802 Forks)auto 404
area/envoyhelp wanted

Repository-Metriken

Stars
 (2.871 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

We should return 421 when http/2 connection coalescing is detected

one workaround for now, is to disable http/2 ALPN when overlapping hostnames are detected, with the OverlappingTLSConfig condition added to the listeners https://gateway-api.sigs.k8s.io/geps/gep-3567/

I might be technically not deep enough into this https://gateway-api.sigs.k8s.io/geps/gep-3567/ states an option b of returning 421 for this. Is there a reason that envoy gateway is doing that which i am missing and/or could this be tracked as a followup in this or a new issue? Ideally it would be nice to not break http2 support like this :)

Originally posted by @MTRNord in #2675

Contributor Guide

Research-Richtung
Untersuche die Envoy Gateway Codebasis, um überlappende Hostnamen in HTTP/2 Verbindungen zu erkennen und gemäß Option b von GEP 3567 den Statuscode 421 zurückzugeben. Verfolge die Verbindungszusammenführungslogik in der Envoy Konfiguration oder der Gateway Implementierung.
Tech Stack
go
Domain
backendnetworkingapi
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GoKubernetesEnvoy
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40