Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@henryju
Copy link
Member

@henryju henryju commented Nov 28, 2025

No description provided.

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title Replace okhttp by the native JDK HttpClient SCANJLIB-287 Replace okhttp by the native JDK HttpClient Nov 28, 2025
@hashicorp-vault-sonar-prod
Copy link

hashicorp-vault-sonar-prod bot commented Nov 28, 2025

SCANJLIB-287

@henryju henryju force-pushed the jh/migrate_from_okhttp_to_jdk branch 2 times, most recently from 8fca049 to 6249e72 Compare November 28, 2025 14:08
@henryju henryju force-pushed the jh/migrate_from_okhttp_to_jdk branch from 46b331a to 98274f9 Compare December 2, 2025 11:04
@henryju henryju marked this pull request as ready for review December 2, 2025 11:04
@sonarqube-next
Copy link

sonarqube-next bot commented Dec 2, 2025

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good🎉, I ran an analysis with local scanner cli, embedding this version of the library, with no issues.
Just for heads-up, while trying to do that, I got this error message from the enforcer plugin in sonar-scanner-cli (when running mvn verify) I assume due to the now-smaller java-library size, so we will probably need to update the config of the plugin when upgrading the version of the library.
Screenshot 2025-12-02 at 15 31 23

@henryju henryju merged commit 931c7ab into master Dec 2, 2025
13 checks passed
@henryju henryju deleted the jh/migrate_from_okhttp_to_jdk branch December 2, 2025 14:52
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.

2 participants