envoyproxy/envoy

Envoy should register invalid HTTP as an attempt to send a request

Open

#9821 aperta il 24 gen 2020

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/httphelp wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

Title: Envoy should register invalid HTTP as an attempt to send a request

Description: If a malformed HTTP1 request is sent (e.g. No headers, just "a\r\n\r\n") the HTTP1.1 code will reject it (https://github.com/envoyproxy/envoy/blob/master/source/common/http/http1/codec_impl.cc#L497) and send a 400 without creating a ConnectionManagerImpl::ActiveStream. This means that the stats around total requests (downstream_rq_total_ & similar) will not be modified, as they are modified by ConnectionManagerImpl::ActiveStream. It seems like an invalid request should still be counted in those statistics.

Guida contributor

Direzione di ricerca
Il problema è che le richieste HTTP/1.1 non valide (ad esempio, intestazioni malformate) vengono rifiutate in source/common/http/http1/codec impl.cc intorno alla riga 497 senza incrementare downstream rq total e le statistiche correlate. La correzione dovrebbe comportare l'aggiunta di un incremento delle statistiche in quel punto, prima di inviare la risposta 400. Controlla la classe ConnectionManagerImpl::ActiveStream per capire come le statistiche vengono normalmente incrementate per le richieste valide.
Tech stack
cpp
Dominio
backendobservability
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++HTTP
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
55