SslHandlerFactory is currently a @FunctionalInterface, which leads to a lot of code duplication.

      Looking at the state of affairs we have ended up after the initial conversion, we need to refactor it.

      SslHandlerFactory should be an abstract class, which provide an internal API to bind SslHandler. The public API is exposed via a protected method which returns an SslContext – and internal take care of creating the actual SslHandler.

      This leads to friction with SslHandlerFactoryProvider, which operates in terms of an internally-created KeyStore and then instantiates SSLEngine – and then wraps it into a SslHandler.

      The provider needs to be thoroughly refactored, so that we use a SslContextBuilder and populate it with the trust chains and private keys/certificate chains. We should hold on to this context for as long as allowedKeys do not change and reuse across devices.

            rovarga Robert Varga
            rovarga Robert Varga
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: