RFR: 8343395: SSLLogger doesn't work for formatted messages [v5]
Sean Coffey
coffeys at openjdk.org
Thu Sep 18 15:39:57 UTC 2025
On Thu, 18 Sep 2025 09:23:15 GMT, Sean Coffey <coffeys at openjdk.org> 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:
>
> Correct old typo
thanks Weijun. I'd missed that point. Yes- I've already tested the solution where we test for "instanceof SSLConsoleLogger" - seems to work fine, I've updated the test case also.
wondering if the System Logger should attempt to print to a similar format. Will get back to you on this.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/25934#issuecomment-3308174304
More information about the security-dev
mailing list