On Tue, 28 Apr 2026 17:17:14 GMT, Rajan Halade <[email protected]> wrote:
>> Mikhail Yankelevich has updated the pull request with a new target base due
>> to a merge or a rebase. The incremental webrev excludes the unrelated
>> changes brought in by the merge/rebase. The pull request contains two
>> additional commits since the last revision:
>>
>> - Merge branch 'master' into JDK-8372817 and update copyright
>> - JDK-8372817: Remove test/jdk/sun/security/rsa silent skips
>
> 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.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28580#discussion_r3160325853
