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