envoyproxy/envoy

grpc: grpc-status in trailers does not trigger retries

Open

#17,629 建立於 2021年8月9日

在 GitHub 查看
 (5 留言) (0 反應) (0 負責人)C++ (5,373 fork)batch import
area/grpcarea/routerhelp wanted

倉庫指標

Star
 (27,997 star)
PR 合併指標
 (平均合併 8天) (30 天內合併 378 個 PR)

描述

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?

貢獻者指南

研究方向
該問題報告Envoy的重試機制僅檢測headers中的grpc status，但當伺服器設定自定義header時，grpc status會出現在trailers中，導致重試失敗。相關程式碼位於source/common/router/router.cc的第1392-1412行附近，以及第1238-1240行的註解。修復可能涉及修改重試邏輯，以同時檢查trailers中的grpc status。研究gRPC規範中關於trailers中grpc status的規定，並審查Envoy中現有的重試實作。一種潛在方法是在回應完成後緩衝trailers並檢查grpc status，然後在需要時觸發重試。
技術棧
cpp
領域
backend
議題類型
錯誤
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
2
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
半天
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
活躍
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
清晰
前置要求
GitC++
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
65