envoyproxy/envoy

Custom headers are not available to the SNI `override_auto_sni_header` when added using `request_headers_to_add`

Open

#18.308 aperta il 29 set 2021

Vedi su GitHub
 (3 commenti) (1 reazione) (0 assegnatari)C++ (5373 fork)batch import
area/httparea/tlshelp wanted

Metriche repository

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

Descrizione

Title

Custom headers are not available to the SNI override_auto_sni_header when added using request_headers_to_add

Description

We now allow a customer header to populate the SNI for the upstream requests [Link]. But, if the custom header is not coming from downstream and gets added anywhere using request_headers_to_add then it's value is not available here as this piece of code to populate the SNI gets invoked before we add/modify the request headers probably here.

Guida contributor

Direzione di ricerca
Indaga l'ordine delle operazioni in router.cc: in particolare, guarda le righe 517 e 648 per capire la sequenza di aggiunta delle intestazioni e popolamento SNI. Determina se spostare il popolamento SNI dopo l'aggiunta delle intestazioni risolve il problema.
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.
1-3 ore
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
Envoy internalsC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
20