envoyproxy/envoy

Remove tokens from the default bucket only if there is no matching descriptor.

Open

#28 469 ouverte le 18 juil. 2023

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/ratelimithelp wanted

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

Title: Remove tokens from the default bucket only if there is no matching descriptor.

Description: I have a question about the Local RateLimit plugin. In version 1.23.0, the local_ratelimit started removing tokens from all matching buckets, including the default bucket, which now always has a token removed. Is there a way to prevent this from happening, removing tokens from the default only if there is no matching descriptor?

If there is no way to do this currently, could we make it parameterizable?

Thank you in advance.

[optional Relevant Links:] https://github.com/envoyproxy/envoy/pull/20869

Guide contributeur

Direction de recherche
La demande concerne un paramètre pour contrôler si les jetons sont retirés du compartiment par défaut uniquement lorsqu'aucun descripteur correspondant n'existe. Reportez vous à la PR liée #20869 qui a modifié le comportement. Étudiez le code source du plugin de limitation de débit local dans le dépôt Envoy, en particulier la logique du compartiment à jetons, pour comprendre comment les descripteurs et le compartiment par défaut interagissent. Proposez une option de configuration pour rétablir l'ancien comportement.
Stack technique
ccpp
Domaine
backendinfrastructure
Type d'issue
Fonctionnalité
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.
Une demi journée
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.
Plutôt claire
Prérequis
C++Envoy
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30