envoyproxy/envoy

Protecting envoy from stacking events on the same fd

Open

#8 924 ouverte le 7 nov. 2019

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/connectionhelp wantedtech debt

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

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

Guide contributeur

Direction de recherche
Étudiez le modèle d'empilement d'événements sur le même descripteur de fichier tel que décrit dans la PR #8922. Définissez un modèle recommandé pour toutes les utilisations d'événements dans Envoy, développez un mécanisme de détection (éventuellement des assertions à l'exécution ou une analyse statique) et créez des outils de débogage tels que le suivi d'événements ou la journalisation pour identifier les violations. Passez en revue l'utilisation existante des événements dans la base de code et documentez les résultats.
Stack technique
cpp
Domaine
backend
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.
1-2 jours
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.
Plutôt claire
Prérequis
C++libevent
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
25