envoyproxy/envoy

Fix inconsistency in rate limit filter names

Open

#8 799 ouverte le 29 oct. 2019

Voir sur GitHub
 (6 commentaires) (0 réactions) (1 assigné)C++ (5 373 forks)batch import
area/ratelimitbeginnerhelp wantedtech debt

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

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)

Guide contributeur

Direction de recherche
Inspectez les configurations existantes du filtre de limitation de débit pour les filtres HTTP et réseau dans les fichiers sources, puis suivez les étapes décrites : enregistrez une seconde définition avec le nouveau nom, dépréciez l'ancienne fabrique de configuration, et après la période de dépréciation, supprimez la. Envisagez également d'étendre cela pour mettre à jour tous les noms de filtres réseau vers le nouveau format.
Stack technique
cpp
Domaine
networking
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70