sermant-io / Sermant

CNCF sandbox project, a Cloud-Native Proxyless Service Mesh based on Java Bytecode Enhancement Technology
https://sermant.io/
Apache License 2.0
1.25k stars 164 forks source link

fix the issue that xds routing fails when okhttp2.2.0 is used #1651

Closed daizhenyu closed 2 weeks ago

daizhenyu commented 3 weeks ago

What type of PR is this?

Bug.

What this PR does / why we need it?

fix the issue that xds routing fails when okhttp2.2.0 is used. The cause is that the okhttp2 earlier version uses java.net.URL instead of com.squareup.okhttp.HttpUrl.

Which issue(s) this PR fixes?

Fixes #1647

Does this PR introduce a user-facing change?

No

Checklist

codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 57.14286% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...eptor/OkHttpClientInterceptorChainInterceptor.java 57.14% 3 Missing :warning:
Flag Coverage Δ Complexity Δ
unittests 44.20% <57.14%> (-0.01%) 181.00 <0.00> (ø)

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

Files with missing lines Coverage Δ Complexity Δ
...eptor/OkHttpClientInterceptorChainInterceptor.java 72.09% <57.14%> (-3.59%) 0.00 <0.00> (ø)

... and 1 file with indirect coverage changes