envoyproxy/envoy

grpc: grpc-status in trailers does not trigger retries

Open

#17.629 aperta il 9 ago 2021

Vedi su GitHub
 (5 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/grpcarea/routerhelp wanted

Metriche repository

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

Descrizione

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?

Guida contributor

Direzione di ricerca
Il problema segnala che i tentativi di ripetizione di Envoy rilevano solo grpc status negli header, ma quando il server imposta un header personalizzato, grpc status appare nei trailer, causando il fallimento dei tentativi. Il codice rilevante si trova in source/common/router/router.cc intorno alle righe 1392-1412 e nel commento alle righe 1238-1240. La correzione probabilmente comporta la modifica della logica di ripetizione per controllare anche i trailer per grpc status. Indagare le specifiche gRPC per grpc status nei trailer e rivedere l'implementazione esistente dei tentativi in Envoy. Un approccio potenziale è memorizzare i trailer in buffer e controllare grpc status dopo il completamento della risposta, quindi attivare il tentativo se necessario.
Tech stack
cpp
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65