envoyproxy/envoy

Connection Limit filter reporting 18 quintillion active connections

Open

#22.127 geöffnet am 12. Juli 2022

Auf GitHub ansehen
 (7 Kommentare) (1 Reaktion) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/ratelimitbughelp wanted

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie die Implementierung des Connection Limit Filters im Envoy Quellcode (wahrscheinlich unter source/extensions/filters/http/connection limit oder ähnlich). Das Problem meldet, dass der active connections Zähler auf 2^64 überläuft. Suchen Sie nach Verwendungen von uint64 t oder ähnlichen Typen in der Zählerlogik des Filters und stellen Sie sicher, dass sie ordnungsgemäß begrenzt sind. Überprüfen Sie die Kommentare in Issue 22127 auf weitere Analysen oder vorgeschlagene Fix Orte.
Tech Stack
cpp
Domain
backendapiinfrastructure
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++Envoy
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40