envoyproxy/envoy

Use response_headers_to_add for Envoy-generated error responses

Open

#8944 aperta il 8 nov 2019

Vedi su GitHub
 (2 commenti) (2 reazioni) (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

On events like a request timeout, Envoy sends a locally generated response (the sendLocalReply function). But it doesn't use the response_headers_to_add config option. Which could be useful when response_headers_to_add has some diagnostic information (e.g. the %REQ(x-request-id)% which was added in #8683)

We can fix this by adding finalizeResponseHeaders calls before sendLocalReply calls, in functions like onUpstreamAbort.

Guida contributor

Direzione di ricerca
Indaga nella codebase di Envoy per individuare le funzioni che chiamano `sendLocalReply` per generare risposte di errore (ad es. `onUpstreamAbort`). Esamina la funzione `finalizeResponseHeaders` per capire come applica `response headers to add`. Aggiungi chiamate a `finalizeResponseHeaders` prima di `sendLocalReply` in tali funzioni. Controlla la PR correlata #8683 per il contesto su come `response headers to add` viene utilizzato per gli header di richiesta.
Tech stack
cpp
Dominio
backendinfrastructure
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
C++Envoy
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60