envoyproxy/envoy

Fix inconsistency in rate limit filter names

Open

#8.799 aberto em 29 de out. de 2019

Ver no GitHub
 (6 comments) (0 reactions) (1 assignee)C++ (5.373 forks)batch import
area/ratelimitbeginnerhelp wantedtech debt

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

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)

Guia do colaborador

Direção de pesquisa
Inspecione as configurações existentes do filtro de limite de taxa para filtros HTTP e de rede nos arquivos de origem, em seguida siga as etapas descritas: registre uma segunda definição com o novo nome, descontinue a fábrica de configuração antiga e, após o período de descontinuação, remova a. Considere também estender isso para atualizar todos os nomes de filtros de rede para o novo formato.
Pilha de tecnologia
cpp
Domain
networking
Tipo Issue
Refatoração
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.
1-3 horas
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.
Claro
Prerequisites
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70