Skip to content

Commit

Permalink
okhttp: Don't warn about missing Conscrypt
Browse files Browse the repository at this point in the history
When running on the JDK, it is quite normal for Conscrypt not to be
present. We'll end up using the JDK 9 ALPN API and everything will be
fine. On Android, it would be extremely rare for someone to completely
remove the default Android security providers, so the warning was almost
never going to trigger on that platform anyway.
  • Loading branch information
ejona86 committed Oct 2, 2024
1 parent 959060a commit 6f35422
Showing 1 changed file with 1 addition and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -283,7 +283,7 @@ private static boolean isAtLeastAndroid41() {

/**
* Select the first recognized security provider according to the preference order returned by
* {@link Security#getProviders}. If a recognized provider is not found then warn but continue.
* {@link Security#getProviders}.
*/
private static Provider getAndroidSecurityProvider() {
Provider[] providers = Security.getProviders();
Expand All @@ -295,7 +295,6 @@ private static Provider getAndroidSecurityProvider() {
}
}
}
logger.log(Level.WARNING, "Unable to find Conscrypt");
return null;
}

Expand Down

0 comments on commit 6f35422

Please # to comment.