[GitHub] [pulsar] lhotari commented on issue #10372: [Performance] Accelerate Jetty HTTPS performance by configuring Jetty to use Conscrypt / OpenSSL provider, also fixes mutual TLS auth on JDK11

[GitBox <gi...@apache.org>]

lhotari commented on issue #10372:
URL: https://github.com/apache/pulsar/issues/10372#issuecomment-826652124


   It seems that switching to use Conscrypt would require more changes than just enabling the Security provider. Quite a few tests (for example org.apache.pulsar.client.api.AuthenticatedProducerConsumerTest) fail  with this type of exception:
   
   ```
   java.lang.IllegalArgumentException: Private key must be accompanied by certificate chain
     	at java.base/java.security.KeyStore.setKeyEntry(KeyStore.java:1170)
     	at org.apache.pulsar.common.util.KeyStoreHolder.setPrivateKey(KeyStoreHolder.java:60)
     	at org.apache.pulsar.common.util.SecurityUtility.setupKeyManager(SecurityUtility.java:259)
     	at org.apache.pulsar.common.util.SecurityUtility.createSslContext(SecurityUtility.java:247)
     	at org.apache.pulsar.common.util.SecurityUtility.createSslContext(SecurityUtility.java:152)
     	at org.apache.pulsar.common.util.DefaultSslContextBuilder.update(DefaultSslContextBuilder.java:48)
     	at org.apache.pulsar.common.util.DefaultSslContextBuilder.update(DefaultSslContextBuilder.java:27)
     	at org.apache.pulsar.common.util.SslContextAutoRefreshBuilder.get(SslContextAutoRefreshBuilder.java:79)
     	at org.apache.pulsar.common.util.SecurityUtility$SslContextFactoryWithAutoRefresh.getSslContext(SecurityUtility.java:452)
     	at org.eclipse.jetty.util.ssl.SslContextFactory.newSSLEngine(SslContextFactory.java:1897)
     	at org.eclipse.jetty.server.SslConnectionFactory.doStart(SslConnectionFactory.java:99)
     	at org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:73)
     	at org.eclipse.jetty.util.component.ContainerLifeCycle.start(ContainerLifeCycle.java:169)
     	at org.eclipse.jetty.util.component.ContainerLifeCycle.doStart(ContainerLifeCycle.java:117)
     	at org.eclipse.jetty.server.AbstractConnector.doStart(AbstractConnector.java:321)
     	at org.eclipse.jetty.server.AbstractNetworkConnector.doStart(AbstractNetworkConnector.java:81)
     	at org.eclipse.jetty.server.ServerConnector.doStart(ServerConnector.java:234)
     	at org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:73)
     	at org.eclipse.jetty.server.Server.doStart(Server.java:401)
     	at org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:73)
     	at org.apache.pulsar.broker.web.WebService.start(WebService.java:238)
     	... 40 more
   ```
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org