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