envoyproxy/envoy

ext_proc: knob to disallow immediate_response

Aberta

#28.698 aberto em 28 de jul. de 2023

Ver no GitHubGuia do colaborador
 (8 comentários) (0 reação) (2 responsáveis)C++ (5.373 forks)batch import
area/ext_procenhancementhelp wanted

Métricas do repositório

Stars
 (27.997 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Title: ext_proc: knob to disallow immediate_response

Description: Just like the way we have a knob disallow_clear_route_cache that will ignore a clear_route_cache setting in the ProcessingResponse, we need a knob on the filter (disallow_immediate_response) that will ignore an immediate_response value in the ProcessingResponse. The behavior should be to close the ext_proc stream but NOT do the immediate response. Instead, the request should be forwarded as through no changes were done by ext_proc.

Knob is false by default.

Guia do colaborador

Direção de pesquisa
Comece estudando a implementação existente do botão `disallow clear route cache` em `source/extensions/filters/http/ext proc/config.cc` e `ext proc.proto`. Entenda como `immediate response` é atualmente tratado na lógica do filtro. O novo botão deve se comportar de forma semelhante: adicione um campo booleano ao proto de configuração do filtro, verifique o no código de processamento de resposta e feche o fluxo sem enviar a resposta imediata. Revise os testes relevantes em `test/extensions/filters/http/ext proc/` para garantir o comportamento correto.
Pilha de tecnologia
cpp
Domain
infrastructureapi
Tipo Issue
Funcionalidade
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-2 dias
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.
50