envoyproxy/envoy

Fix inconsistency in rate limit filter names

Open

#8.799 geöffnet am 29. Okt. 2019

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (5.373 Forks)batch import
area/ratelimitbeginnerhelp wantedtech debt

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

Minor cleanup related to PR #8796, for HTTP filters we have envoy.rate_limit but for networking filters we have envoy.ratelimit. 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 of the rate limit filter's config factory with the new name.
  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)

Contributor Guide

Research-Richtung
Überprüfen Sie die vorhandenen Konfigurationen des Ratenbegrenzungsfilters für HTTP und Netzwerkfilter in den Quelldateien und befolgen Sie dann die beschriebenen Schritte: registrieren Sie eine zweite Definition mit dem neuen Namen, stellen Sie die alte Konfigurationsfabrik als veraltet ein und entfernen Sie sie nach der Abkündigungsfrist. Erwägen Sie auch, dies zu erweitern, um alle Netzwerkfilternamen auf das neue Format zu aktualisieren.
Tech Stack
cpp
Domain
networking
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70