Skip to content

Commit 6f35422

Browse files
committed
okhttp: Don't warn about missing Conscrypt
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.
1 parent 959060a commit 6f35422

File tree

1 file changed

+1
-2
lines changed
  • okhttp/third_party/okhttp/main/java/io/grpc/okhttp/internal

1 file changed

+1
-2
lines changed

okhttp/third_party/okhttp/main/java/io/grpc/okhttp/internal/Platform.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -283,7 +283,7 @@ private static boolean isAtLeastAndroid41() {
283283

284284
/**
285285
* Select the first recognized security provider according to the preference order returned by
286-
* {@link Security#getProviders}. If a recognized provider is not found then warn but continue.
286+
* {@link Security#getProviders}.
287287
*/
288288
private static Provider getAndroidSecurityProvider() {
289289
Provider[] providers = Security.getProviders();
@@ -295,7 +295,6 @@ private static Provider getAndroidSecurityProvider() {
295295
}
296296
}
297297
}
298-
logger.log(Level.WARNING, "Unable to find Conscrypt");
299298
return null;
300299
}
301300

0 commit comments

Comments
 (0)