On Fri, 19 Sep 2025 12:08:14 GMT, Sean Coffey <[email protected]> wrote:

>> SLSLogger is broken when used with with `System.Logger` (-Djavax.net.debug 
>> mode)
>> 
>> SSL Debug messages don't use format specifiers. As a result, any custom 
>> format data isn't printed. Proposed solution is to append the SSL custom 
>> format output to the original debug message. Similar approach used for 
>> `SSLConsoleLogger ` mode.
>> 
>> Used this opportunity to delete some old commented code and to replace use 
>> of `"\n"` with `System.lineSeparator()`
>> 
>> DebugPropertyValuesTest also updated to test new logic.
>
> Sean Coffey has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Avoid Supplier API call

Marked as reviewed by weijun (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/25934#pullrequestreview-3245803599

Reply via email to