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

Mikhail Yankelevich myankelevich at openjdk.org
Fri Oct 17 15:57:08 UTC 2025


On Fri, 17 Oct 2025 14:34:57 GMT, Artur Barashev <abarashev at openjdk.org> wrote:

>> I agree with logging this as warning. However I think keeping `KeyMgr:` would be better, as this format is used in the existing code in the class. What do you think?
>
> Oh, I see. Let's remove the `KeyMgr: ` prefix throughout the whole file, it was left here from the initial version of this file when logging was done by a simple debug logger. So such prefix was useful then, but not now. Also let's make the log message more informative: `"Exception thrown while getting an alias " + alias + ": " + e`

Ok, thanks. Done in the next commit

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

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


More information about the security-dev mailing list