[jdk17u] RFR: 8263031: HttpClient throws Exception if it receives a Push Promise that is too large

Martin Balao mbalao at openjdk.org
Thu Sep 26 07:01:00 UTC 2024


Hi,

Can I have a review for the backport of JDK-8263031 to 17u? This backport fixes a bug and will also serve as a dependency for a backport of 8328286.

The JDK main line patch did not apply cleanly because JDK-8295944 was backported to 17u already and tests were moved.

In addition, I had to make changes to PushPromiseContinuation.java to adapt the test to its new location (see jtreg header diff) and context (see the invocations to conn.addToOutputQ). With these changes, the test is passing.

No regressions observed in the jdk/java/net/httpclient tests category.

Thanks,
Martin.-

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

Commit messages:
 - Backport 4d2cd26ab5092ad0a169e4239164a869a4255bd3

Changes: https://git.openjdk.org/jdk17u/pull/400/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk17u&pr=400&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8263031
  Stats: 446 lines in 4 files changed: 406 ins; 19 del; 21 mod
  Patch: https://git.openjdk.org/jdk17u/pull/400.diff
  Fetch: git fetch https://git.openjdk.org/jdk17u.git pull/400/head:pull/400

PR: https://git.openjdk.org/jdk17u/pull/400


More information about the jdk-updates-dev mailing list