Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

流量标签透传特性:支持httpclient4.x #1251

Merged
merged 1 commit into from
Jul 18, 2023

Conversation

lilai23
Copy link
Collaborator

@lilai23 lilai23 commented Jul 17, 2023

【修复issue】#1244

【修改内容】

1、新增流量标签透传插件
2、支持httpclient4.x客户端的流量标签透传

【用例描述】后续补充测试用例

【自测情况】1、本地静态检查通过;2、本地自测通过

【影响范围】无

@codecov
Copy link

codecov bot commented Jul 17, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.06 ⚠️

Comparison is base (3fb63d0) 41.52% compared to head (8de452d) 41.46%.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #1251      +/-   ##
=============================================
- Coverage      41.52%   41.46%   -0.06%     
  Complexity      3212     3212              
=============================================
  Files            817      820       +3     
  Lines          18032    18056      +24     
  Branches        1961     1963       +2     
=============================================
  Hits            7487     7487              
- Misses          9607     9631      +24     
  Partials         938      938              
Flag Coverage Δ
unittests 41.46% <0.00%> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...uaweicloud/sermant/core/utils/CollectionUtils.java 0.00% <0.00%> (ø)
...a/com/huaweicloud/sermant/core/utils/MapUtils.java 60.00% <0.00%> (-4.29%) ⬇️
...huaweicloud/sermant/core/utils/tag/TrafficTag.java 0.00% <0.00%> (ø)
...aweicloud/sermant/core/utils/tag/TrafficUtils.java 0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@lilai23 lilai23 force-pushed the tag_trans branch 2 times, most recently from 42016d9 to a572e89 Compare July 18, 2023 02:31
provenceee
provenceee previously approved these changes Jul 18, 2023
@Sherlockhan Sherlockhan merged commit 8cdb26b into sermant-io:develop Jul 18, 2023
66 checks passed
@luanwenfei-venus luanwenfei-venus changed the title 【feature】流量标签透传特性:支持httpclient4.x 流量标签透传特性:支持httpclient4.x Jul 18, 2023
@luanwenfei-venus luanwenfei-venus added kind/feature Issue for new feature area/plugin Issues or PRs releated to sermant plugin service labels Jul 18, 2023
@luanwenfei-venus luanwenfei-venus added this to the v1.2.0 milestone Jul 18, 2023
@hanbingleixue hanbingleixue removed this from the v1.2.0 milestone Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/plugin Issues or PRs releated to sermant plugin service kind/feature Issue for new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants