shouldLogRequestHeader and shouldLogResponseHeader is not enough to control headers log in FULL mode
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
issue でリンクされているレスポンスロギングの行がある core/src/main/java/feign/Logger.java から始め、現在レスポンスヘッダーがどのようにフィルタリングされているかを追跡してください。FULL モードのレスポンスボディログに含まれるヘッダーも shouldLogResponseHeader に従い、選択された機密ヘッダーがそこに表示されないようになれば、変更は完了です。
索引モデルが issue の本文から書いたものです。
説明
There is an implementation to control request and response headers with shouldLogRequestHeader and shouldLogResponseHeader.
It's working for logs but …
There is a line in the implementation which logs the whole response body also with headers.
I would also like to control these headers. Filter out particular headers in that particular log.
I am using FULL logging and debug mode.
The reason why I filter out the headers is to avoid security sensitive headers like tokens etc.
I would like to see that shouldLogResponseHeader is controlling also this line of the log too without any other implementation.
Here is how the log looks like
-10-04T10:36:50.206+02:00 DEBUG 131409 --- [demo] [ main] feign.Logger : [Client#get] {
"args": {},
"headers": {
"Accept": "*/*",
"Host": "[httpbin.org](http://httpbin.org/)",
"User-Agent": "Java/21.0.1",
"X-Amzn-Trace-Id": "Root=1-66ffa922-324f4c83482b70b4579853ac"
},
"origin": "88.212.19.236",
"url": "https://httpbin.org/get"
}
- 主要言語
- Java
- スター
- 9.8k
- フォーク
- 2k
- 平均マージ
- 1時間 23分
- マージ済み PR（30日）
- 31
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
OpenFeign/feign のほかの issue
-
DefaultAsyncClient never completes the returned future when Client throws Error対応中かも @hutiefang76 が 20 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
Response.charset ignores charset after another Content-Type parameter対応中かも @hutiefang76 が 20 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
UriUtils ignores allowReservedCharacters=false when the input has no percent triplet対応中かも @hutiefang76 が 20 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
Java11 Http2Client loses response header value order and duplicate occurrences対応中かも @hutiefang76 が 20 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 35/100
メンテナーはふだん 1 日以内に返信
-
Java11 Http2Client rejects documented zero connect and read timeouts対応中かも @hutiefang76 が 20 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 35/100
メンテナーはふだん 1 日以内に返信
OpenFeign/feign の issue をすべて見る
似ている issue
-
finsim-core good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
commons-app/apps-android-commons#7275 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
sonatype-nexus-community/central-publishing-maven-plugin#13 ·
-
documentation
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
liberation-data/drivine4j#682 ·
メンテナーはふだん 1 日以内に返信
-
Document and enforce pull request review guidelines対応中かも @dashorst が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信