envoyproxy/envoy

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

Open

#28.469 aperta il 18 lug 2023

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/ratelimithelp wanted

Metriche repository

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

Descrizione

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

Guida contributor

Direzione di ricerca
La richiesta riguarda un parametro per controllare se i token vengono rimossi dal bucket predefinito solo quando non esiste un descrittore corrispondente. Fare riferimento alla PR collegata #20869 che ha modificato il comportamento. Indagare il codice sorgente del plugin di limitazione della velocità locale nel repository Envoy, in particolare la logica del bucket dei token, per comprendere come i descrittori e il bucket predefinito interagiscono. Proporre un'opzione di configurazione per ripristinare il comportamento precedente.
Tech stack
ccpp
Dominio
backendinfrastructure
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
C++Envoy
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30