envoyproxy/envoy

Avoid using HeaderMapImpl for gRPC metadata

开放

#7,921 创建于 2019年8月14日

在 GitHub 查看贡献者指南
 (1 条评论) (0 个反应) (0 位负责人)C++ (5,373 个派生)batch import
area/securitybughelp wanted

仓库指标

星标
 (27,997 个星标)
PR 合并指标
 (PR 指标待抓取)

描述

Today, we use HeaderMapImpl for gRPC metadata, e.g. https://github.com/envoyproxy/envoy/blob/3dd84d3b41b6be378ccb71e81c0063a84bb956e3/include/envoy/grpc/async_client.h#L70. This is not safe, since in general, this metadata may contain binary and HeaderMapImpl can't contain NUL.

We should switch to using a simple map or the like to model these values, treating them as metadata rather than headers.

贡献者指南

研究方向
找出所有在gRPC元数据中使用HeaderMapImpl的地方，并将其替换为支持二进制键和值的容器，例如std::map或absl::flat hash map。
技术栈
cpp
领域
networking
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
C++gRPC
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
30