envoyproxy/envoy

Deprecate Http2ProtocolOptions.allow_connect

Open

#10.412 aperta il 17 mar 2020

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

Metriche repository

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

Descrizione

PR #9964 introduced user defined HTTP/2 SETTINGS parameters and disallowed overriding "named" parameters (e.g., hpack_table_size) with these custom parameters. However, an exception had to be made for allow_connect, a field for which presence can't be checked due to the use of a scalar type.

An API change must be introduced to remove the exception and enable uniformity in override handling.

Guida contributor

Direzione di ricerca
Indaga l'uso di allow connect in Http2ProtocolOptions e rimuovi la gestione speciale per questo campo per consentire l'override uniforme delle impostazioni denominate.
Tech stack
cpp
Dominio
backend
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65