envoyproxy/envoy

Router: Audit ASSERTs

Open

#14.961 aperta il 5 feb 2021

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/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 router filter and recommendations to change some: https://gist.github.com/asraa/859a41a8f5f252d35da58c9ef7502230

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

Guida contributor

Direzione di ricerca
Rivedi la gist all'indirizzo https://gist.github.com/asraa/859a41a8f5f252d35da58c9ef7502230, che descrive l'audit degli ASSERT. Il file principale da modificare è source/common/router/router.cc, con le righe specifiche: 276, 566 e 653. Il compito prevede l'aggiunta di un controllo per gli header dello schema durante l'elaborazione del filtro e l'audit di altri ASSERT come indicato. Esamina il codice attorno a quelle righe per comprendere la logica corrente degli assert e applicare le modifiche consigliate. Non esistono PR collegati, quindi l'implementazione è indipendente.
Tech stack
cpp
Dominio
backendinfrastructure
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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 internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40