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

Consistent use of SSLHandshakeException for ALPN

Browse Source 
Motiviation:
The OpenSSL engine uses SSLHandshakeException in the event of failures that occur during the handshake process. The alpn-boot project's getSSLException will also map the no_application_protocol to a SSLHandshakeException exception. We should be consistent and use SSLHandshakeException for handshake failure events.

Modifications:
-Update JdkAlpnSslEngine to propagate an SSLHandshakeException in the event of a failure.

Result:
Consistent usage of SSLHandshakeException during a handshake failure event.
This commit is contained in:
Scott Mitchell 2015-05-22 14:05:39 -07:00
parent 4cac79594c
commit 0ce7c3b2a0
1 changed files with 16 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
handler/src/main/java/io/netty/handler/ssl/JdkAlpnSslEngine.java
View File

@ -16,17 +16,19 @@
package io.netty.handler.ssl;
import static io.netty.util.internal.ObjectUtil.checkNotNull;
import io.netty.handler.ssl.JdkApplicationProtocolNegotiator.ProtocolSelectionListener;
import io.netty.handler.ssl.JdkApplicationProtocolNegotiator.ProtocolSelector;
import org.eclipse.jetty.alpn.ALPN;
import org.eclipse.jetty.alpn.ALPN.ClientProvider;
import org.eclipse.jetty.alpn.ALPN.ServerProvider;
import java.util.LinkedHashSet;
import java.util.List;
import javax.net.ssl.SSLEngine;
import javax.net.ssl.SSLException;
import java.util.LinkedHashSet;
import java.util.List;
import javax.net.ssl.SSLHandshakeException;
import org.eclipse.jetty.alpn.ALPN;
import org.eclipse.jetty.alpn.ALPN.ClientProvider;
import org.eclipse.jetty.alpn.ALPN.ServerProvider;
final class JdkAlpnSslEngine extends JdkSslEngine {
private static boolean available;
@ -63,12 +65,12 @@ final class JdkAlpnSslEngine extends JdkSslEngine {
public String select(List<String> protocols) throws SSLException {
try {
return protocolSelector.select(protocols);
} catch (SSLException e) {
} catch (SSLHandshakeException e) {
throw e;
} catch (Throwable t) {
// Ensure that all exceptions are propagated as SSLExceptions
// so that the SslHandler properly fails the handshake.
throw new SSLException(t);
SSLHandshakeException e = new SSLHandshakeException(t.getMessage());
e.initCause(t);
throw e;
}
}
@ -91,12 +93,12 @@ final class JdkAlpnSslEngine extends JdkSslEngine {
public void selected(String protocol) throws SSLException {
try {
protocolListener.selected(protocol);
} catch (SSLException e) {
} catch (SSLHandshakeException e) {
throw e;
} catch (Throwable t) {
// Ensure that all exceptions are propagated as SSLExceptions
// so that the SslHandler properly fails the handshake.
throw new SSLException(t);
SSLHandshakeException e = new SSLHandshakeException(t.getMessage());
e.initCause(t);
throw e;
}
}