On Mon, 22 Sep 2025 18:42:57 GMT, Mark Powers <[email protected]> wrote:

>> [JDK-8343232](https://bugs.openjdk.org/browse/JDK-8343232)
>
> Mark Powers has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   default salt length and one other comment from Weijun

src/java.base/share/classes/sun/security/pkcs12/PKCS12KeyStore.java line 2208:

> 2206:                     } else {
> 2207: 
> 2208:                         // Change SHA-1 to SHA1

While you are in this code, can you update this comment? It's not only SHA-1, 
it's any message digest with a "-" in it, so suggest changing it to something 
like:

`// Remove "-" from digest algorithm names`

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24429#discussion_r2373286486

Reply via email to