[jdk11u-dev] RFR: 8309841: Jarsigner should print a warning if an entry is removed [v2]
Andrew John Hughes
andrew at openjdk.org
Thu Mar 20 02:06:14 UTC 2025
On Mon, 10 Mar 2025 08:30:59 GMT, Roman Marchenko <rmarchenko at openjdk.org> wrote:
>> This is backport of "8309841: Jarsigner should print a warning if an entry is removed"
>>
>> Original patch does not apply cleanly to jdk11, some minor conflicts, and also in `test/jdk/sun/security/tools/jarsigner/RemovedFiles.java` `ed25519` algorithm was replaced with `RSA`, and `_` was replaced with a variable, and added the check in `src/jdk.jartool/share/classes/sun/security/tools/jarsigner/Main.java` at line 1259.
>>
>> We need this fix in jdk11 and below, down to jdk8, as all versions have this issue with jarsigner.
>>
>> New tests successfully ran locally on Linux, x86_64.
>
> Roman Marchenko has updated the pull request incrementally with one additional commit since the last revision:
>
> Fixing review comments
Backport looks good. The different `hasNonexistentEntries` handling seems consistent with the lack of [JDK-8250586](https://bugs.openjdk.org/browse/JDK-8250586) in 11u. EdDSA support was only added in OpenJDK 16 by [JDK-8242068](https://bugs.openjdk.org/browse/JDK-8242068).
I couldn't see the `_` replacement mentioned. Was this backport based on the 17u version?
-------------
Marked as reviewed by andrew (Reviewer).
PR Review: https://git.openjdk.org/jdk11u-dev/pull/3007#pullrequestreview-2700905813
More information about the jdk-updates-dev
mailing list