Skip to content

fix: http协议客户端对请求头的处理和服务端解析不一致#132

Open
sppan24 wants to merge 1 commit intotrpc-group:masterfrom
sppan24:master
Open

fix: http协议客户端对请求头的处理和服务端解析不一致#132
sppan24 wants to merge 1 commit intotrpc-group:masterfrom
sppan24:master

Conversation

@sppan24
Copy link
Contributor

@sppan24 sppan24 commented Mar 25, 2026

http协议客户端发起请求的时候,没有按照"tRPC请求报文头映射"的约定设置请求头,导致客户端发起请求后,服务端或者其他语言的服务端无法从请求头获取到必要的数据(例如框架透传信息等)。

@codecov
Copy link

codecov bot commented Mar 25, 2026

Codecov Report

❌ Patch coverage is 95.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.61905%. Comparing base (8ba3d46) to head (625c748).

Files with missing lines Patch % Lines
...t/trpc/proto/http/client/Http2ConsumerInvoker.java 95.65217% 1 Missing ⚠️
...nt/trpc/proto/http/client/HttpConsumerInvoker.java 94.11765% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@                  Coverage Diff                  @@
##                master        #132         +/-   ##
=====================================================
+ Coverage     85.59593%   85.61905%   +0.02312%     
  Complexity        4307        4307                 
=====================================================
  Files              436         436                 
  Lines            14364       14401         +37     
  Branches          1287        1296          +9     
=====================================================
+ Hits             12295       12330         +35     
- Misses            2069        2071          +2     
Files with missing lines Coverage Δ
...t/trpc/proto/http/client/Http2ConsumerInvoker.java 89.89899% <95.65217%> (+1.29139%) ⬆️
...nt/trpc/proto/http/client/HttpConsumerInvoker.java 98.70130% <94.11765%> (-1.29870%) ⬇️

... and 7 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant