envoyproxy/envoy

Connection Limit filter reporting 18 quintillion active connections

Open

#22.127 aberto em 12 de jul. de 2022

Ver no GitHub
 (7 comments) (1 reaction) (0 assignees)C++ (5.373 forks)batch import
area/ratelimitbughelp 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

We have observed that Envoy with the Connection Limit Filter configured, at some point will report 2**64 active connections for the metric envoy.connection_limit.<listener>.active_connections.

This seems suspicious and could be indicative of some integer overflow.

This did not result in a crash, but did result in some connections being limited when it was not intended.

We configured the filter with a max_connections of 56000 and delay of 500ms

Guia do colaborador

Direção de pesquisa
Investigue a implementação do filtro Connection Limit no código fonte do Envoy (provavelmente em source/extensions/filters/http/connection limit ou similar). O problema relata que o contador active connections estoura para 2^64. Procure por usos de uint64 t ou tipos semelhantes na lógica do contador do filtro e certifique se de que eles sejam devidamente limitados. Verifique os comentários na issue 22127 para qualquer análise adicional ou locais de correção propostos.
Pilha de tecnologia
cpp
Domain
backendapiinfrastructure
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.
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.
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.
40