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

feat: added functionality to choose between apache and okHttp for sending HTTP requests via settings #1132

Merged
merged 1 commit into from
May 12, 2024

Conversation

tangcent
Copy link
Owner

ref: #1130

@tangcent tangcent added the type: new feature Add New Feature label Apr 28, 2024
@tangcent tangcent force-pushed the feature/https branch 2 times, most recently from 4460d00 to c7e29aa Compare April 28, 2024 23:55
Copy link

codecov bot commented Apr 29, 2024

Codecov Report

Attention: Patch coverage is 81.68498% with 50 lines in your changes are missing coverage. Please review.

Project coverage is 62.697%. Comparing base (0f706c1) to head (e0fa630).
Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff               @@
##              master     #1132       +/-   ##
===============================================
+ Coverage     62.073%   62.697%   +0.624%     
- Complexity      2289      2372       +83     
===============================================
  Files            250       254        +4     
  Lines          12126     12302      +176     
  Branches        2996      3032       +36     
===============================================
+ Hits            7527      7713      +186     
- Misses          2994      3006       +12     
+ Partials        1605      1583       -22     
Flag Coverage Δ
unittests 62.697% <81.685%> (+0.624%) ⬆️

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

Files Coverage Δ
...main/kotlin/com/itangcent/http/ApacheHttpClient.kt 92.157% <100.000%> (+0.853%) ⬆️
...i/src/main/kotlin/com/itangcent/http/HttpClient.kt 100.000% <ø> (ø)
...m/itangcent/idea/plugin/settings/HttpClientType.kt 100.000% <100.000%> (ø)
...lin/com/itangcent/idea/plugin/settings/Settings.kt 98.225% <100.000%> (+32.038%) ⬆️
...nt/idea/plugin/settings/xml/ApplicationSettings.kt 100.000% <100.000%> (ø)
...m/itangcent/suv/http/AbstractHttpClientProvider.kt 100.000% <100.000%> (ø)
...om/itangcent/suv/http/DefaultHttpClientProvider.kt 100.000% <100.000%> (ø)
.../itangcent/suv/http/HttpClientScriptInterceptor.kt 92.405% <100.000%> (ø)
.../src/main/kotlin/com/itangcent/http/HttpRequest.kt 85.385% <50.000%> (-0.553%) ⬇️
...gcent/idea/plugin/api/export/suv/SuvApiExporter.kt 0.000% <0.000%> (ø)
... and 7 more

... and 4 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0f706c1...e0fa630. Read the comment docs.

@KK-Karl
Copy link

KK-Karl commented Apr 30, 2024

试了 ,真的可以了。
谢谢大神。
112233

@tangcent tangcent force-pushed the feature/https branch 8 times, most recently from fa0f7bb to b8d0bda Compare May 11, 2024 01:04
@tangcent tangcent merged commit c4c77a4 into master May 12, 2024
10 checks passed
@tangcent tangcent deleted the feature/https branch May 12, 2024 00:25
@tangcent tangcent mentioned this pull request May 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: new feature Add New Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants