[jdk8u-dev] RFR: 8309841: Jarsigner should print a warning if an entry is removed [v3]

duke duke at openjdk.org
Thu Mar 27 17:02:09 UTC 2025


On Wed, 26 Mar 2025 15:14:36 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 jdk8:
>> - some minor conflicts in JDK code
>> - files moved to appropriate locations
>> - added the check in `jdk/src/share/classes/sun/security/tools/jarsigner/Main.java` at line 1196
>> 
>> in tests:
>> - `ed25519` algorithm was replaced with `RSA` in `RemovedFiles.java`
>> - `JarEntry copyEntry()` procedure was manually added to `JarUtils.java` to make `JarUtilsTest` working.
>> - the tests were adapted to Java 8, including replaced `Path.of`, `Files.writeString`, package names, and arrays processing as well.
>> 
>> We need this fix in 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 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 three additional commits since the last revision:
> 
>  - Merge branch 'master' into b8309841
>  - Backport bdfb41f977258831e4b0ceaef5d016d095ab6e7f
>  - Backport 2a791467919c9df9869e6fe1e57df0a5caa90d8f

@wkia 
Your change (at version 75504f7cd3cdb7341616e023dd4222be4a686d24) is now ready to be sponsored by a Committer.

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

PR Comment: https://git.openjdk.org/jdk8u-dev/pull/635#issuecomment-2758777114


More information about the jdk8u-dev mailing list