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

Artur Barashev abarashev at openjdk.org
Mon Oct 20 17:22:05 UTC 2025


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

>> Mikhail Yankelevich has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Artur's comments
>
> src/java.base/share/classes/sun/security/ssl/X509KeyManagerImpl.java line 231:
> 
>> 229:             alias.length() - secondDot < 2) {
>> 230: 
>> 231:             if (SSLLogger.isOn && SSLLogger.isOn("keymanager")) {
> 
> `ssl,keymanager`   
> 
> We're in the process of cleaning these up.

I guess we should change it throughout the file then.

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

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


More information about the security-dev mailing list