envoyproxy/envoy

wasm: metrics never expire and accumulate

Open

#14,070 opened on 2020年11月17日

GitHub で見る
 (14 comments) (8 reactions) (0 assignees)C++ (5,373 forks)batch import
area/statsarea/wasmbughelp wanted

Repository metrics

Stars
 (27,997 stars)
PR merge metrics
 (平均マージ 7d 21h) (30d で 260 merged PRs)

説明

Wasm filter implements the stats by allocating Envoy stats in the context and never de-allocating them. Since ABI encourages dynamic values, the stats accumulate and eventually leak (also, potentially exposing a security risk if stat values are untrusted).

コントリビューターガイド

調査方針
Envoy ソースコード内の Wasm フィルター実装を調査します（おそらく source/extensions/filters/http/wasm/ 以下にあります）。Envoy 統計情報が現在 ABI 経由でどのように割り当てられているか、またなぜそれらが期限切れにならないのかを理解します。提案された解決策とメンテナーからのフィードバックについて、issue #14070 の議論を確認します。Wasm モジュールが破棄された時、または定期的なクリーンアップを通じて、統計情報を追跡し解放するメカニズムの実装を検討します。
技術スタック
cppwasm
領域
backendperformancesecurityobservability
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
4
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
3-5日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
古い
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
Envoy proxy basicsWebAssembly modulesEnvoy stat subsystem
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
25