envoyproxy/envoy

Envoy should NACK an EDS update with duplicate addresses

Open

#23.445 aberto em 11 de out. de 2022

Ver no GitHub
 (2 comments) (0 reactions) (1 assignee)C++ (5.373 forks)batch import
area/edsbughelp 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

Currently, if there are multiple addresses in the same EDS resource, Envoy will ignore all but the first occurrance. This behavior seems fairly arbitrary and surprising, especially because the different occurrances of the address could be in different localities or priorities.

I think in this case, Envoy should NACK the EDS update.

CC @kyessenov

Guia do colaborador

Direção de pesquisa
Investigue o código de manipulação de atualização EDS em envoy/source/common/upstream/eds.cc, especificamente a lógica que ignora endereços duplicados. Rastreie o caminho da análise da resposta EDS até a geração do NACK. Entenda a estrutura de validação e como rejeitar uma atualização com endereços duplicados acionando um NACK.
Pilha de tecnologia
None
Domain
infrastructure
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
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
C++Envoy internals
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50