8253952: Refine ZipOutputStream.putNextEntry() to recalculate ZipEntry's compressed size

[vieiro]

Backport of JDK-8253952

Clean backport with a minor conflict resolved. Tests pass cleanly.

I have a test case that shows differences in behaviour between Amazon Corretto 11.0.23.9-1 (built with --zlib=system) and adoptium-jdk-11.0.23+9 (built with --zlib=bundled) on Fedora 40 with system-wide zlib-ng. Let me know if you think it's worth adding the test case to the codebase.

---

Progress

• Change must be properly reviewed (1 review required, with at least 1 Reviewer)
• JDK-8253952 needs maintainer approval
• Change must not contain extraneous whitespace
• Commit message must refer to an issue
• Change requires a CSR request matching fixVersion 11.0.25 to be approved (needs to be created)

Issue

• JDK-8253952: Refine ZipOutputStream.putNextEntry() to recalculate ZipEntry's compressed size (Enhancement - P4)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk11u-dev.git pull/2765/head:pull/2765
$ git checkout pull/2765

Update a local copy of the PR:
$ git checkout pull/2765
$ git pull https://git.openjdk.org/jdk11u-dev.git pull/2765/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 2765

View PR using the GUI difftool:
$ git pr show -t 2765

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk11u-dev/pull/2765.diff

Webrev

Link to Webrev Comment

[bridgekeeper]

👋 Welcome back vieiro! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

[openjdk]

❗ This change is not yet ready to be integrated.
See the Progress checklist in the description for automated requirements.

[openjdk]

This backport pull request has now been updated with issue from the original commit.

[openjdk]

At least one of the issues associated with this backport has a resolved CSR for a different version. As this means that this backport may also need a CSR, the csr label is being added to this pull request to signal this potential requirement. The command /csr unneeded can be used to remove the label in case a CSR is not needed.

[mlbridge]

Webrevs

• 00: Full (dd50c85f)

[vieiro]

/approval JDK-8253952 request Refine ZipOutputStream.putNextEntry() to recalculate ZipEntry's compressed size

[openjdk]

@vieiro
JDK-8253952: The approval request has been created successfully.

[vieiro]

This PR requires a CSR! I don't think it's worth the hassle for such a small corner-case. Closing as a consequence. Apologies for the trouble.