支持grpc中http2 setting新增扩展字段(GRPC_CHTTP2_SETTINGS_GRPC_ALLOW_TRUE_BINARY_METADATA)
- Dominant language
- C++
- Stars
- 17.6k
- Forks
- 4.1k
- Avg merge
- 2d 12h
- Merged PRs (30d)
- 69
Description
1. grpc增加了对http2 协议settings字段的扩展,[具体介绍](https://github.com/grpc/proposal/blob/master/G1-true-binary-metadata.md),目前新接入的grpc都会带有此扩展,希望能支持此功能。
2. brpc中解析函数处[http2_rpc_protocol.cpp](https://github.com/apache/incubator-brpc/blob/master/src/brpc/policy/http2_rpc_protocol.cpp):207行行会输出大量的warning日志,是否考虑可以删除?
附[RFC7540](https://datatracker.ietf.org/doc/html/rfc7540#section-6.5.2)
Contributor guide
Research direction
Start with src/brpc/policy/http2_rpc_protocol.cpp around line 207, then read the linked GRPC true-binary-metadata proposal and RFC 7540 section 6.5.2. Trace how HTTP/2 SETTINGS fields are parsed and determine the expected handling for GRPC_CHTTP2_SETTINGS_GRPC_ALLOW_TRUE_BINARY_METADATA. Confirm the warning behavior and whether it should be removed, with tests covering the supported setting.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp, grpc
- Domain
- backend-api-design, networking
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100