envoyproxy/envoy

grpc: grpc-status in trailers does not trigger retries

Open

#17.629 aberto em 9 de ago. de 2021

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
area/grpcarea/routerhelp 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

I noticed that currently for gRPC:

Envoy's retries will only detect grpc-status in Headers.When the server sets a custom Header, grpc-status will appear in the Trailers, which will cause Envoy's retries to not work: https://github.com/grpc/grpc-go/blob/master/stream.go#L1369

I think we should do something in here: https://github.com/envoyproxy/envoy/blob/main/source/common/router/router.cc#L1392-L1412

I noticed this comment, which makes me think that the developers didn't accidentally overlook the problem, but just put it aside for now: https://github.com/envoyproxy/envoy/blob/main/source/common/router/router.cc#L1238-L1240

So are there any special considerations or difficulties? Will I get support if I submit a PR to fix this issue?

Guia do colaborador

Direção de pesquisa
O problema relata que as tentativas de repetição do Envoy detectam apenas grpc status nos cabeçalhos, mas quando o servidor define um cabeçalho personalizado, grpc status aparece nos trailers, causando falha nas repetições. O código relevante está em source/common/router/router.cc nas linhas 1392-1412 e no comentário nas linhas 1238-1240. A correção provavelmente envolve modificar a lógica de repetição para também verificar os trailers em busca de grpc status. Investigue a especificação gRPC para grpc status em trailers e revise a implementação de repetição existente no Envoy. Uma abordagem potencial é armazenar os trailers em buffer e verificar grpc status após a conclusão da resposta, então acionar a repetição, se necessário.
Pilha de tecnologia
cpp
Domain
backend
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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
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
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65