envoyproxy/envoy

Use response_headers_to_add for Envoy-generated error responses

Open

#8.944 aberto em 8 de nov. de 2019

Ver no GitHub
 (2 comments) (2 reactions) (0 assignees)C++ (5.373 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Investigue a base de código do Envoy em busca de funções que chamam `sendLocalReply` para gerar respostas de erro (por exemplo, `onUpstreamAbort`). Examine a função `finalizeResponseHeaders` para entender como ela aplica `response headers to add`. Adicione chamadas a `finalizeResponseHeaders` antes de `sendLocalReply` nessas funções. Consulte o PR relacionado #8683 para contexto sobre como `response headers to add` é usado para cabeçalhos de solicitação.
Pilha de tecnologia
cpp
Domain
backendinfrastructure
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Envoy
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60