envoyproxy/envoy

Add support of additional %UPSTREAM% variables to custom headers

Open

#13.897 aperta il 4 nov 2020

Vedi su GitHub
 (5 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/httpdesign proposalhelp wanted

Metriche repository

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

Descrizione

One of our customers is using the Envoy tap filter to retrieve transport information and add them to their request/response headers, as documented here : https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_conn_man/headers.html#custom-request-response-headers

At this point, very few %UPSTREAM% variable names are supported : %UPSTREAM_METADATA([“namespace”, “key”, …])% %UPSTREAM_REMOTE_ADDRESS%

They wish additional %UPSTREAM% variables were supported, such as %UPSTREAM_LOCAL_ADDRESS%.

Thanks !

Guida contributor

Direzione di ricerca
Esamina il codice di espansione delle variabili esistente in Envoy (probabilmente in source/common/http/header formatter.cc o simile) per capire come vengono gestite le attuali variabili %UPSTREAM ...%, quindi aggiungi il supporto per nuove variabili come %UPSTREAM LOCAL ADDRESS% seguendo lo stesso schema.
Tech stack
Nessuno
Dominio
backendinfrastructure
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65