diff --git a/networking/src/main/java/google/registry/networking/handler/SslServerInitializer.java b/networking/src/main/java/google/registry/networking/handler/SslServerInitializer.java index 5be1af53a..dc0525fca 100644 --- a/networking/src/main/java/google/registry/networking/handler/SslServerInitializer.java +++ b/networking/src/main/java/google/registry/networking/handler/SslServerInitializer.java @@ -113,8 +113,6 @@ public class SslServerInitializer extends ChannelInitializer< sslHandler.engine().getSession().getPeerCertificates()[0]; try { clientCertificate.checkValidity(); - Promise unusedPromise = - clientCertificatePromise.setSuccess(clientCertificate); } catch (CertificateNotYetValidException | CertificateExpiredException e) { logger.atWarning().withCause(e).log( "Client certificate is not valid.\nHash: %s", @@ -123,8 +121,11 @@ public class SslServerInitializer extends ChannelInitializer< Promise unusedPromise = clientCertificatePromise.setFailure(e); ChannelFuture unusedFuture2 = channel.close(); + return; } } + Promise unusedPromise = + clientCertificatePromise.setSuccess(clientCertificate); } else { Promise unusedPromise = clientCertificatePromise.setFailure(future.cause());