On Tue, 3 Sep 2024 17:38:30 GMT, Rajan Halade <[email protected]> wrote:
>> Mark Powers has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> beware moving lines around
>
> test/jdk/sun/security/ssl/X509TrustManagerImpl/Entrust/Distrust.java line 113:
>
>> 111: return xtm;
>> 112: }
>> 113: throw new Exception("No TrustManager for " + type);
>
> Please update this and other `Exception` thrown in `loadCertificateChain` and
> `testTM` function to `RuntimeException`.
Will address with separate bug.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20731#discussion_r1742607064