andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Clarify exception message when ALPN is not supported (#10155)

Browse Source 
Motivation:

We should provide more informations when ALPN is not supported and a user tries to use it.

Modifications:

- Use UnsupportedOperationException

Result:

Easier to debug ALPN problems
This commit is contained in:
Norman Maurer 2020-04-22 08:07:27 +02:00 committed by GitHub
parent 5cd00d22d9
commit d0c2f6e8b3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
handler/src/main/java/io/netty/handler/ssl/JdkAlpnApplicationProtocolNegotiator.java
View File

@ -144,7 +144,8 @@ public final class JdkAlpnApplicationProtocolNegotiator extends JdkBaseApplicati
return isServer ? JettyAlpnSslEngine.newServerEngine(engine, applicationNegotiator) return isServer ? JettyAlpnSslEngine.newServerEngine(engine, applicationNegotiator)
: JettyAlpnSslEngine.newClientEngine(engine, applicationNegotiator); : JettyAlpnSslEngine.newClientEngine(engine, applicationNegotiator);
} }
throw new RuntimeException("Unable to wrap SSLEngine of type " + engine.getClass().getName()); throw new UnsupportedOperationException("ALPN not supported. Unable to wrap SSLEngine of type '"
+ engine.getClass().getName() + "')");
} }
} }