envoyproxy/envoy

Use response_headers_to_add for Envoy-generated error responses

Open

#8,944 创建于 2019年11月8日

在 GitHub 查看
 (2 评论) (2 反应) (0 负责人)C++ (5,373 fork)batch import
enhancementhelp wanted

仓库指标

Star
 (27,997 star)
PR 合并指标
 (平均合并 8天) (30 天内合并 378 个 PR)

描述

On events like a request timeout, Envoy sends a locally generated response (the sendLocalReply function). But it doesn't use the response_headers_to_add config option. Which could be useful when response_headers_to_add has some diagnostic information (e.g. the %REQ(x-request-id)% which was added in #8683)

We can fix this by adding finalizeResponseHeaders calls before sendLocalReply calls, in functions like onUpstreamAbort.

贡献者指南

研究方向
研究 Envoy 代码库中调用 `sendLocalReply` 生成错误响应的函数（例如 `onUpstreamAbort`）。检查 `finalizeResponseHeaders` 函数以了解其如何应用 `response headers to add`。在这些函数中，于 `sendLocalReply` 之前添加对 `finalizeResponseHeaders` 的调用。参考相关 PR #8683 以了解 `response headers to add` 如何用于请求头。
技术栈
cpp
领域
backendinfrastructure
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
C++Envoy
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
60