On Thu, 31 Oct 2024 20:28:06 GMT, Alan Bateman <[email protected]> wrote:
>> 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.
Added comment.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21565#discussion_r1826153929
