False positive: Env var is from config, not vault, and contains the name of another env var
まだ誰も着手していません。
評価
調査の方向性
crates/config/src/embeddings.rs の 76-79 行目にあるリンク先のコードから始め、code-scanning アラート 5068 を調査して、検出結果を生成している CodeQL クエリとデータフローパスを特定します。この設定値が秘密情報の出力として報告されなくなり、同時に本物の環境変数の秘密情報の漏えいは引き続き検出されれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Description of the false positive
This flagged for outputting the value of an environment variable to logs. Generally, that could be a problem. In this case, the env var clearly contained the name of another env var to look in for the secret. Is there a way to not flag in this situation? For example, could we determine that this environment variable came from a k8s env var (where secrets are not allowed) as opposed to from vault?
Code samples or links to source code
URL to the alert on GitHub code scanning (optional)
https://github.com/github/blackbird/security/code-scanning/5068
- 主要言語
- CodeQL
- スター
- 10.1k
- フォーク
- 2.1k
- 平均マージ
- 2日 10時間
- マージ済み PR（30日）
- 134
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
github/codeql のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
-
C#: cs/simplifiable-boolean-expression false positive on Nullable<bool> compared with a literal オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
false-positive
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
false-positive
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
似ている issue
-
Product: Azure Policy :shield: Topic: Diagnostic Settings :test_tube: Topic: Policy :pencil:
難易度 1/5 1〜3時間 初心者へのやさしさ 92/100
Azure/Azure-Landing-Zones#4283 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
MystenLabs/MemWal#979 · コメント 1 件 ·
-
namespace operations
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
EclipseFdn/open-vsx.org#13384 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
-
documentation
難易度 2/5 半日 初心者へのやさしさ 62/100
inmanta/inmanta-core#10835 ·