Skip to content

fix: multiple clients conflicts when clients have proxies#166

Merged
yndu13 merged 1 commit intomasterfrom
fix-clients-conflict
Mar 9, 2023
Merged

fix: multiple clients conflicts when clients have proxies#166
yndu13 merged 1 commit intomasterfrom
fix-clients-conflict

Conversation

@yndu13
Copy link
Contributor

@yndu13 yndu13 commented Mar 9, 2023

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Mar 9, 2023

Codecov Report

Merging #166 (221948e) into master (acc041a) will increase coverage by 0.14%.
The diff coverage is 85.71%.

❗ Current head 221948e differs from pull request most recent head 5874c6d. Consider uploading reports for the commit 5874c6d to get more accurate results

@@             Coverage Diff              @@
##             master     #166      +/-   ##
============================================
+ Coverage     88.13%   88.28%   +0.14%     
- Complexity      400      404       +4     
============================================
  Files            27       27              
  Lines          1079     1084       +5     
  Branches        159      161       +2     
============================================
+ Hits            951      957       +6     
  Misses           83       83              
+ Partials         45       44       -1     
Flag Coverage Δ
java 88.28% <85.71%> (+0.14%) ⬆️

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

Impacted Files Coverage Δ
.../main/java/com/aliyun/tea/okhttp/ClientHelper.java 96.29% <85.71%> (-3.71%) ⬇️
...ava/com/aliyun/tea/okhttp/OkHttpClientBuilder.java 86.45% <0.00%> (+2.08%) ⬆️

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

@yndu13 yndu13 merged commit f8adaf8 into master Mar 9, 2023
@yndu13 yndu13 deleted the fix-clients-conflict branch March 9, 2023 04:15
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.

3 participants