envoyproxy/envoy

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

Open

#28.469 aberto em 18 de jul. de 2023

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
area/ratelimithelp wanted

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
A questão solicita um parâmetro para controlar se os tokens são removidos do bucket padrão apenas quando não existe um descritor correspondente. Consulte o PR vinculado #20869 que alterou o comportamento. Investigue o código fonte do plugin de limitação de taxa local no repositório do Envoy, particularmente a lógica do bucket de tokens, para entender como os descritores e o bucket padrão interagem. Proponha uma opção de configuração para reverter para o comportamento antigo.
Pilha de tecnologia
ccpp
Domain
backendinfrastructure
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
C++Envoy
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30