envoyproxy / envoyproxy/envoy

Fix inconsistency in rate limit filter names

オープン
#8,799 コメント 6 件 リアクション 0 件 担当者 1 名 @arvpo が担当を希望しています GitHub で見る
area/ratelimit beginner help wanted tech debt
主要言語
C++
スター
28.9k
フォーク
5.6k
平均マージ
1日 22時間
マージ済み PR(30日)
430

説明

Minor cleanup related to PR #8796, for HTTP filters we have [`envoy.rate_limit`](https://github.com/envoyproxy/envoy/blob/master/source/extensions/filters/http/well_known_names.h#L39) but for networking filters we have [`envoy.ratelimit`](https://github.com/envoyproxy/envoy/blob/master/source/extensions/filters/network/well_known_names.h#L27). We should update the network filter to match HTTP/Thrift. In order to avoid naming collisions, the network filter should be updated to the new style of `envoy.filters.network.rate_limit`. This would involve
1. [Register a second definition](https://github.com/envoyproxy/envoy/blob/master/source/extensions/filters/network/ratelimit/config.cc#L44) of the rate limit filter's [config factory](https://github.com/envoyproxy/envoy/blob/master/source/extensions/filters/network/ratelimit/config.h#L18) with the [new name](https://github.com/envoyproxy/envoy/blob/master/source/extensions/filters/network/ratelimit/config.h#L21).
2. Deprecation period for the old config factory
3. Removal of the old config factory

(while here, we could extend this to updating all the networking filter names to the new format, this one just stood out as a potential nuisance)

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

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

評価

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

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

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