sofastack / sofa-rpc-node

SOFARPC Node is a high-performance, high-extensibility, production-level Nodejs RPC framework.
MIT License
612 stars 64 forks source link

chore: add parseRequestPropsWithHeader decode option #104

Closed mytEcust closed 1 year ago

mytEcust commented 1 year ago

https://github.com/sofastack/sofa-rpc/blob/master/remoting/remoting-bolt/src/main/java/com/alipay/sofa/rpc/codec/bolt/SofaRpcSerialization.java#L325 对齐 sofa-bolt 的实现,通过 parseRequestPropsWithHeader 的 decode option 配置 header 中的 rpc_trace_context 属性合并进 req

codecov[bot] commented 1 year ago

Codecov Report

Merging #104 (997d650) into master (6929c71) will decrease coverage by 0.04%. The diff coverage is 50.00%.

:exclamation: Current head 997d650 differs from pull request most recent head 5bbe5d6. Consider uploading reports for the commit 5bbe5d6 to get more accurate results

@@            Coverage Diff             @@
##           master     #104      +/-   ##
==========================================
- Coverage   98.27%   98.22%   -0.05%     
==========================================
  Files          38       38              
  Lines        2313     2315       +2     
  Branches      368      369       +1     
==========================================
+ Hits         2273     2274       +1     
- Misses         40       41       +1     
Impacted Files Coverage Δ
lib/server/server.js 100.00% <ø> (ø)
lib/server/connection.js 96.25% <50.00%> (-1.19%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

killagu commented 1 year ago

+ sofa-rpc-node@2.5.0