apache / apache/brpc

支持grpc中http2 setting新增扩展字段(GRPC_CHTTP2_SETTINGS_GRPC_ALLOW_TRUE_BINARY_METADATA)

Open
#1,579 0 comments 0 reactions 0 assignees View on GitHub
feature
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.