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