On Thu, 31 Oct 2024 19:59:00 GMT, Dean Long <[email protected]> wrote:
>> Patricio Chilano Mateo has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Fix typos in comments
>
> src/java.base/share/classes/sun/security/ssl/X509TrustManagerImpl.java line
> 57:
>
>> 55: static {
>> 56: try {
>> 57:
>> MethodHandles.lookup().ensureInitialized(AnchorCertificates.class);
>
> Why is this needed? A comment would help.
That's probably a good idea. It’s caused by pinning due to the
sun.security.util.AnchorCertificates’s class initializer, some of the http
client tests are running into this. Once monitors are out of the way then class
initializers, both executing, and waiting for, will be a priority.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21565#discussion_r1825127591
