envoyproxy/envoy

To add LocalInfoFormatter and LocalInfoHeaderFormatter

Open

#7213 aperta il 8 giu 2019

Vedi su GitHub
 (0 commenti) (1 reazione) (0 assegnatari)C++ (5373 fork)batch import
enhancementhelp wanted

Metriche repository

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

Descrizione

To add LocalInfoFormatter and LocalInfoHeaderFormatter.

Description: Currently seems like there is no way to log information holds by LocalInfo while the information is sometimes useful and also to utilize that information as values in e.g. response_headers_to_add, request_headers_to_add.

Probably adding LocalInfoHeaderFormatter and LocalInfoFormatter in header formatter and access log formatter respectively will help.

@zuercher WDYT?

Guida contributor

Direzione di ricerca
Esamina le implementazioni esistenti dei formattatori come RequestInfoHeaderFormatter e AccessLogFormatter per comprendere il pattern, quindi implementa LocalInfoFormatter e LocalInfoHeaderFormatter per esporre i campi di LocalInfo.
Tech stack
cpp
Dominio
backend
Tipo issue
Funzionalità
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
C++Envoy proxy internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30