envoyproxy/envoy

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

Open

#28.469 geöffnet am 18. Juli 2023

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/ratelimithelp wanted

Repository-Metriken

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

Beschreibung

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

Contributor Guide

Research-Richtung
Das Issue fordert einen Parameter, um zu steuern, ob Token nur dann aus dem Standard Bucket entfernt werden, wenn kein passender Deskriptor existiert. Siehe den verlinkten PR #20869, der das Verhalten geändert hat. Untersuchen Sie den Quellcode des lokalen Ratenbegrenzungs Plugins im Envoy Repository, insbesondere die Token Bucket Logik, um zu verstehen, wie Deskriptoren und der Standard Bucket interagieren. Schlagen Sie eine Konfigurationsoption vor, um zum alten Verhalten zurückzukehren.
Tech Stack
ccpp
Domain
backendinfrastructure
Issue Type
Funktion
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.
Halber Tag
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.
Meist klar
Voraussetzungen
C++Envoy
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30