envoyproxy/envoy

Protecting envoy from stacking events on the same fd

Open

#8,924 创建于 2019年11月7日

在 GitHub 查看
 (1 评论) (0 反应) (0 负责人)C++ (5,373 fork)batch import
area/connectionhelp wantedtech debt

仓库指标

Star
 (27,997 star)
PR 合并指标
 (平均合并 8天) (30 天内合并 378 个 PR)

描述

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

贡献者指南

研究方向
研究 PR #8922 中描述的在同一文件描述符上堆叠事件的模式。为 Envoy 中的所有事件使用定义一个推荐模式，开发一种检测机制（可能是运行时断言或静态分析），并创建调试工具（例如事件跟踪或日志记录）以识别违规行为。审查代码库中现有的事件使用情况并记录发现。
技术栈
cpp
领域
backend
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-2 天
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
基本清晰
前置要求
C++libevent
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
25