envoyproxy/envoy

Connection Limit filter reporting 18 quintillion active connections

Open

#22,127 opened on 2022年7月12日

GitHub で見る
 (7 comments) (1 reaction) (0 assignees)C++ (5,373 forks)batch import
area/ratelimitbughelp wanted

Repository metrics

Stars
 (27,997 stars)
PR merge metrics
 (平均マージ 8d) (30d で 378 merged PRs)

説明

We have observed that Envoy with the Connection Limit Filter configured, at some point will report 2**64 active connections for the metric envoy.connection_limit.<listener>.active_connections.

This seems suspicious and could be indicative of some integer overflow.

This did not result in a crash, but did result in some connections being limited when it was not intended.

We configured the filter with a max_connections of 56000 and delay of 500ms

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

調査方針
Envoy ソースコード内の Connection Limit フィルターの実装を調査します（おそらく source/extensions/filters/http/connection limit または類似のディレクトリ下）。この問題は、active connections カウンターが 2^64 にオーバーフローすると報告しています。フィルターのカウンターロジックにおける uint64 t または類似の型の使用を探し、適切に境界が設定されていることを確認してください。さらなる分析や提案された修正箇所については、issue 22127 のコメントを確認してください。
技術スタック
cpp
領域
backendapiinfrastructure
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
半日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
C++Envoy
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
40