envoyproxy/envoy

Connection Limit filter reporting 18 quintillion active connections

Open

#22 127 ouverte le 12 juil. 2022

Voir sur GitHub
 (7 commentaires) (1 réaction) (0 assignés)C++ (5 373 forks)batch import
area/ratelimitbughelp wanted

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

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

Guide contributeur

Direction de recherche
Étudiez l'implémentation du filtre Connection Limit dans le code source d'Envoy (probablement sous source/extensions/filters/http/connection limit ou similaire). Le problème signale que le compteur active connections déborde vers 2^64. Recherchez les utilisations de uint64 t ou de types similaires dans la logique du compteur du filtre et assurez vous qu'ils sont correctement bornés. Vérifiez les commentaires dans l'issue 22127 pour toute analyse supplémentaire ou emplacements de correctifs proposés.
Stack technique
cpp
Domaine
backendapiinfrastructure
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Envoy
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40