On Wed, 29 Apr 2026 10:42:13 GMT, Mikhail Yankelevich 
<[email protected]> wrote:

>> test/jdk/sun/security/rsa/pss/SignatureTest2.java line 184:
>> 
>>> 182:             if (params == null) {
>>> 183:                 skippedAlgs.add(
>>> 184:                         String.format("[digestAlg: %s, digestLen: %d, 
>>> " +
>> 
>> In your test run, did you see this skip happen? This is an intentional skip. 
>> If you intend to keep this as skipped then update the log message on line 
>> 189 to include `digestAlg`, `digestLen`, `keySize` values to co-relate with 
>> skipped exception.
>
> It doesn't skip on any platform for me. I'm happy to add the details in the 
> next commit.

Actually, the skipped algorithms are logged in the end as well, but I still 
think it's best logging them in 2 spots

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28580#discussion_r3160365558

Reply via email to