<html><head><meta http-equiv="Content-Type" content="text/html; charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Done..<div class=""><br class=""></div><div class="">ID: <span style="caret-color: rgb(51, 51, 51); color: rgb(51, 51, 51); font-family: "Helvetica Neue", "Segoe UI", Roboto, Helvetica, Arial, sans-serif; font-size: 14px; background-color: rgb(255, 255, 255);" class="">9057278</span></div><div class=""><font color="#333333" face="Helvetica Neue, Segoe UI, Roboto, Helvetica, Arial, sans-serif" class=""><span style="caret-color: rgb(51, 51, 51); font-size: 14px; background-color: rgb(255, 255, 255);" class=""><br class=""></span></font></div><div class=""><font color="#333333" face="Helvetica Neue, Segoe UI, Roboto, Helvetica, Arial, sans-serif" class=""><span style="caret-color: rgb(51, 51, 51); font-size: 14px; background-color: rgb(255, 255, 255);" class="">Thanks</span></font></div><div class=""><font color="#333333" face="Helvetica Neue, Segoe UI, Roboto, Helvetica, Arial, sans-serif" class=""><span style="caret-color: rgb(51, 51, 51); font-size: 14px; background-color: rgb(255, 255, 255);" class="">Norman</span></font></div><div class=""><font color="#333333" face="Helvetica Neue, Segoe UI, Roboto, Helvetica, Arial, sans-serif" class=""><span style="caret-color: rgb(51, 51, 51); font-size: 14px; background-color: rgb(255, 255, 255);" class=""><br class=""></span></font><div><br class=""><blockquote type="cite" class=""><div class="">On 17. Sep 2018, at 18:51, Xuelei Fan <<a href="mailto:xuelei.fan@oracle.com" class="">xuelei.fan@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">Hi Norman,<br class=""><br class="">Please file an issue for the tracking.<br class=""><br class="">Thanks,<br class="">Xuelei<br class=""><br class="">On 9/17/2018 5:57 AM, Norman Maurer wrote:<br class=""><blockquote type="cite" class="">Should I open an issue somewhere to keep track of it or will you take care of it ?<br class="">Bye<br class="">Norman<br class=""><blockquote type="cite" class="">On 11. Sep 2018, at 17:01, Norman Maurer <<a href="mailto:norman.maurer@googlemail.com" class="">norman.maurer@googlemail.com</a>> wrote:<br class=""><br class="">This sounds great.<br class=""><br class="">I have no idea how many people still use X509TrustManager, sorry.<br class=""><br class="">It may be a good idea to add something to the java docs to tell people to prefer X509ExtendedTrustManager as well.<br class=""><br class="">Bye<br class="">Norman<br class=""><br class=""><blockquote type="cite" class="">Am 11.09.2018 um 16:44 schrieb Xuelei Fan <<a href="mailto:xuelei.fan@oracle.com" class="">xuelei.fan@oracle.com</a>>:<br class=""><br class="">Hi Norman,<br class=""><br class=""><br class="">It may be doable by adding a delegation mode to public TrustManagerFactory:<br class=""> TrustManagerFactory.init(X509TrustManager proxy)<br class=""><br class="">However, the X509ExtendedTrustManager should be recommended for now since its introducing in JDK 7.<br class=""><br class="">Do you know how many users are still using the X509TrustManager implementation?<br class=""><br class="">Thanks,<br class="">Xuelei<br class=""><br class=""><blockquote type="cite" class="">On 9/11/2018 3:32 AM, Norman Maurer wrote:<br class="">Hi all,<br class="">Would it be possible to consider exposing SSLContextImpl#AbstractTrustManagerWrapper somehow so it would be possible to reuse it when a custom SSLEngine / SSLContextSpi is provided ?<br class="">I am asking because it provides really nice extra functionality by wrapping for X509TrustManager implementation and do extra hostname checks etc. At the moment we can not make use of this extra functionality in netty with our custom SSLEngine implementation as there is no way to access this. Which means depending on if the user use our implementation or the default implementation the behaviour if quite different when using a X509TrustManager in the sense that when using the default implementation a lot of extra checks are done.<br class="">As the extra checks done in AbstractTrustManagerWrapper is not really depending on the underlying SSLContextSpi implementation (at least as far as I was able to understand it so far) it would be nice to be able to make use of it.<br class="">Bye<br class="">Norman<br class=""></blockquote></blockquote></blockquote></blockquote></div></div></blockquote></div><br class=""></div></body></html>