Skip to content
New issue

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

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

Already on GitHub? # to your account

[http-client] Fix NPE on attempt to execute HTTP request with URI having invalid protocol #5217

Conversation

valfirst
Copy link
Collaborator

No description provided.

@valfirst valfirst requested a review from a team as a code owner July 15, 2024 21:07
Copy link

Qodana for JVM

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

Copy link

codecov bot commented Jul 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.48%. Comparing base (a82b2b9) to head (15a7889).

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5217      +/-   ##
============================================
+ Coverage     97.47%   97.48%   +0.01%     
  Complexity     6856     6856              
============================================
  Files           947      947              
  Lines         19812    19812              
  Branches       1309     1308       -1     
============================================
+ Hits          19311    19313       +2     
+ Misses          391      390       -1     
+ Partials        110      109       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

@valfirst valfirst merged commit 6df2444 into master Jul 15, 2024
9 checks passed
@valfirst valfirst deleted the fix-npe-on-attempt-to-execute-http-request-with-invalid-protocol branch July 15, 2024 22:37
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant