RFR: 8369995: Implement extra logging and/or propagate errors in X509KeyManagerImpl and fix StringIndexOutOfBoundsException [v6]

Mikhail Yankelevich myankelevich at openjdk.org
Mon Oct 20 19:28:05 UTC 2025


On Mon, 20 Oct 2025 16:18:26 GMT, Bradford Wetmore <wetmore at openjdk.org> wrote:

>> test/jdk/sun/security/ssl/X509KeyManager/NullCases.java line 41:
>> 
>>> 39:  *
>>> 40:  * @run junit NullCases
>>> 41:  * @run junit/othervm -Djavax.net.debug=ssl:keymanager -Darg=debug NullCases
>> 
>> Why do we have a 2nd run with debugging on?
>
> I am in the process of disabling extra (unneeded) debug output.  It looks like there is a good reason to run with debug to check the logging output here, but is there a reason to do an initial run **without** the debugging on?

I wanted to cover all potential cases with and without debug on. In case the debug ever breaks

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27851#discussion_r2445900410


More information about the security-dev mailing list