envoyproxy/envoy

Protecting envoy from stacking events on the same fd

Open

#8.925 geöffnet am 7. Nov. 2019

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
help wantedtech debt

Repository-Metriken

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

Beschreibung

Description: The way envoy FileEventImpl use libevent is tricky. If two FileEventImpl refer to the same fd, eventually the assigned events are stacked and it's probably hard to recover the intention of each other.

https://github.com/envoyproxy/envoy/pull/8922 is the one I encountered but I believe there are more hidden.

We need to figure out

  1. what is pattern we should follow all over envoy,
  2. how to detect the violation
  3. what are the tools to debug when we suspect there is violation

Contributor Guide

Research-Richtung
Untersuchen Sie, wie FileEventImpl libevent verwendet, und identifizieren Sie Muster, die zum Stapeln von Ereignissen auf demselben fd führen. Studieren Sie PR #8922, um das aufgetretene Problem zu verstehen. Bestimmen Sie ein konsistentes Muster in Envoy, um dies zu vermeiden, und schlagen Sie Erkennungstools (z. B. statische Analyse oder Laufzeit Assertions) vor, um Verstöße zu kennzeichnen.
Tech Stack
cpp
Domain
backend
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.
3-5 Tage
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++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
25