envoyproxy/envoy

Unify header validation across all codecs (H1, H2 and H3)

Open

#10.646 aperta il 3 apr 2020

Vedi su GitHub
 (27 commenti) (0 reazioni) (1 assegnatario)C++ (5373 fork)batch import
area/httparea/quicarea/securityhelp wantedtech debt

Metriche repository

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

Descrizione

Presently H1 and H2 codecs use header validation which is a mix of the codec specific and some checks from nghttp2 library on top of it. This leads to inconsistencies in header validation across codecs and makes header validation hard to audit.

For more information see design specifications.

This change will include:

  1. Adding header validation according to the HTTP spec.
  2. Converting codecs to use unified header validation facility. This will only be applied for Balsa and oghttp2 codecs.

Deployment plan:

  1. Add opt-in for universal header validation for Balsa and oghttp2 in compatibility mode (fully compatible with the http-parser and nghttp2)
  2. After bake-in interval (i.e. one full release cycle) change the feature to opt-out
  3. Turn-off compatibility features to bring HTTP validation to RFC compliance one by one.

Guida contributor

Direzione di ricerca
Esamina il documento di progettazione e identifica le specifiche incongruenze nella validazione degli header tra i codec H1, H2 e H3. Quindi, implementa la funzionalità di validazione unificata e migra i codec Balsa e oghttp2 come descritto.
Tech stack
cpp
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
4
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 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++HTTPGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30