KTOR-4712 Use Netty's alpn check instead of attempting to load jetty-…

…alpn class (#3115) Prefer SslProvider.OPENSSL over SslProvider.JDK for better performance
ktorio · Sep 2, 2022 · d917744 · d917744
1 parent aab530a
commit d917744
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/ktor-server/ktor-server-netty/jvm/src/io/ktor/server/netty/NettyChannelInitializer.kt b/ktor-server/ktor-server-netty/jvm/src/io/ktor/server/netty/NettyChannelInitializer.kt
@@ -175,14 +175,15 @@ public class NettyChannelInitializer(
 
         private fun findAlpnProvider(): SslProvider? {
             try {
-                Class.forName("sun.security.ssl.ALPNExtension", true, null)
-                return SslProvider.JDK
+                if (SslProvider.isAlpnSupported(SslProvider.OPENSSL)) {
+                    return SslProvider.OPENSSL
+                }
             } catch (ignore: Throwable) {
             }
 
             try {
-                if (SslProvider.isAlpnSupported(SslProvider.OPENSSL)) {
-                    return SslProvider.OPENSSL
+                if (SslProvider.isAlpnSupported(SslProvider.JDK)) {
+                    return SslProvider.JDK
                 }
             } catch (ignore: Throwable) {
             }