envoyproxy/envoy

grpc: grpc-status in trailers does not trigger retries

Open

#17.629 geöffnet am 9. Aug. 2021

Auf GitHub ansehen
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/grpcarea/routerhelp wanted

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

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?

Contributor Guide

Research-Richtung
Das Problem meldet, dass Envoys Wiederholungsversuche nur grpc status in Headern erkennen, aber wenn der Server einen benutzerdefinierten Header setzt, erscheint grpc status in Trailern, was zu fehlgeschlagenen Wiederholungen führt. Der relevante Code befindet sich in source/common/router/router.cc um die Zeilen 1392-1412 und im Kommentar in den Zeilen 1238-1240. Die Lösung beinhaltet wahrscheinlich die Änderung der Wiederholungslogik, um auch Trailer auf grpc status zu überprüfen. Untersuchen Sie die gRPC Spezifikation für grpc status in Trailern und überprüfen Sie die bestehende Wiederholungsimplementierung in Envoy. Ein möglicher Ansatz ist das Puffern von Trailern und das Überprüfen auf grpc status nach Abschluss der Antwort, dann Auslösen der Wiederholung bei Bedarf.
Tech Stack
cpp
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65