vy commented on PR #3902: URL: https://github.com/apache/logging-log4j2/pull/3902#issuecomment-3308567603
@ppkarwasz, thanks for clarifying the use case in mind. That certainly helps with setting the feature's intended context. > Where I _do_ see value, though, is in introducing an **interface** rather than tying everything to the concrete `SslConfiguration` class. I understand and share your preference for an interface, yet given the current state of the code, wouldn't it be less intrusive to provide a similar adapter pattern support by adding a sufficient `SslConfiguration.create(SSLContext, HostnameVerifier, SSLParameters)` factory method? -- 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. To unsubscribe, e-mail: notifications-unsubscr...@logging.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
