RFR: 8371333: Optimize static initialization of SSLContextImpl classes and improve logging [v6]

Sean Coffey coffeys at openjdk.org
Tue Feb 24 12:07:03 UTC 2026


On Mon, 23 Feb 2026 23:32:50 GMT, Hai-May Chao <hchao at openjdk.org> wrote:

>> Sean Coffey has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 11 commits:
>> 
>>  - Clean up sync of DebugPropertyValuesTest.java
>>  - Remove verbose option for now, difficult to exercise code path
>>  - Merge branch 'master' into 8371333-ssl-debug
>>  - Merge branch 'master' into 8371333-ssl-debug
>>  - Incorportate comments from Jamil
>>  - Merge branch 'master' into 8371333-ssl-debug
>>  - Merge branch 'master' into 8371333-ssl-debug
>>  - Move wrapText method to Utilities
>>  - Merge branch 'master' into 8371333-ssl-debug
>>  - use LINE_SEP
>>  - ... and 1 more: https://git.openjdk.org/jdk/compare/a2961293...ed4c5687
>
> test/jdk/sun/security/ssl/SSLLogger/DebugPropertyValuesTest.java line 64:
> 
>> 62:                 List.of("\"logger\".*: \"javax.net.ssl\",",
>> 63:                     "\"specifics\"   : \\[",
>> 64:                     "\"message\".*: \"Produced ClientHello handshake message"));
> 
> It looks like we could keep the original code as is with 8-space indent.

Hi @haimaychao  - line 64 was extending past the recommended 80 char limit. That's something that Brad likes to keep in check so I pulled these lines in a bit.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28511#discussion_r2846596666



More information about the security-dev mailing list