envoyproxy/envoy

Fix inconsistency in rate limit filter names

Open

#8799 aperta il 29 ott 2019

Vedi su GitHub
 (6 commenti) (0 reazioni) (1 assegnatario)C++ (5373 fork)batch import
area/ratelimitbeginnerhelp wantedtech debt

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

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)

Guida contributor

Direzione di ricerca
Ispeziona le configurazioni esistenti del filtro di limitazione della velocità per i filtri HTTP e di rete nei file sorgente, quindi segui i passaggi indicati: registra una seconda definizione con il nuovo nome, depreca la vecchia factory di configurazione e, dopo il periodo di deprecazione, rimuovila. Considera anche di estendere questo aggiornamento a tutti i nomi dei filtri di rete nel nuovo formato.
Tech stack
cpp
Dominio
networking
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70