envoyproxy / envoyproxy/envoy

Use response_headers_to_add for Envoy-generated error responses

オープン
#8,944 コメント 2 件 リアクション 2 件 担当者 0 名 GitHub で見る
enhancement help wanted
主要言語
C++
スター
28.9k
フォーク
5.6k
平均マージ
1日 22時間
マージ済み PR(30日)
430

説明

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`.

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。