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