RFR: 8279842: HTTPS Channel Binding support for Java GSS/Kerberos [v3]

Michael McMahon michaelm at openjdk.java.net
Fri Jan 21 15:54:59 UTC 2022


On Fri, 21 Jan 2022 13:39:06 GMT, Michael Osipov <duke at openjdk.java.net> wrote:

>> Actually, it turns out I should be throwing `NamingException` here. That is what was being thrown by `TlsChannelBinding.parseType` before and an existing test was expecting that. NamingException only takes a String message. So, there won't be a root cause exception.
>
> `NamingException` has `setRootCause()`. Why not use that? I use that one too and full stack is retained.

Yes, I can do that. Though it will cause the existing LDAP channel binding test to fail which is checking for an empty root cause. That is checking unspecified behavior and I can change it to check for a `ChannelBindingException` as root cause. So long as we are okay having a non public exception type as the root cause, it's probably helpful to have the full stack there.

-------------

PR: https://git.openjdk.java.net/jdk/pull/7065


More information about the core-libs-dev mailing list