envoyproxy/envoy

grpc: grpc-status in trailers does not trigger retries

Open

#17 629 ouverte le 9 août 2021

Voir sur GitHub
 (5 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/grpcarea/routerhelp wanted

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

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?

Guide contributeur

Direction de recherche
Le problème signale que les nouvelles tentatives d'Envoy ne détectent que grpc status dans les en têtes, mais lorsque le serveur définit un en tête personnalisé, grpc status apparaît dans les trailers, provoquant l'échec des nouvelles tentatives. Le code pertinent se trouve dans source/common/router/router.cc autour des lignes 1392-1412 et dans le commentaire aux lignes 1238-1240. La correction implique probablement de modifier la logique de nouvelle tentative pour vérifier également les trailers pour grpc status. Étudiez la spécification gRPC pour grpc status dans les trailers et examinez l'implémentation existante des nouvelles tentatives dans Envoy. Une approche potentielle consiste à mettre en mémoire tampon les trailers et à vérifier grpc status après la fin de la réponse, puis à déclencher une nouvelle tentative si nécessaire.
Stack technique
cpp
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65