envoyproxy/envoy

HTTP/1.1 codec: Audit ASSERTs

Open

#14.959 aperta il 5 feb 2021

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/httparea/securityhelp wanted

Metriche repository

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

Descrizione

This issue describes a scan of ASSERTs in the HTTP/1.1 codec and recommendations to change some: https://gist.github.com/asraa/9d5d60b0af53dc88ce39d8cfd71ef12a

Those marked with a verdict have been audited or unknown if they need a pair of eyes.

Guida contributor

Direzione di ricerca
Rivedi i risultati dell'audit al link https://gist.github.com/asraa/9d5d60b0af53dc88ce39d8cfd71ef12a. Quindi implementa le modifiche per le due condizioni attivabili alle righe 341 e 381 in source/common/http/http1/codec impl.cc. Per le righe contrassegnate come UNKNOWN (141, 804, 1148), analizza le asserzioni per determinare se possono essere attivate da input non attendibili; in tal caso, convertile in una gestione degli errori appropriata. I maintainer non hanno fornito ulteriori chiarimenti nei commenti.
Tech stack
cpp
Dominio
backend
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++Envoy development setup
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40