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

Martin Balao mbalao at openjdk.org
Thu Sep 26 20:46:21 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.-

Martin Balao has updated the pull request incrementally with one additional commit since the last revision:

  Remove unnecessary @modules header.

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

Changes:
  - all: https://git.openjdk.org/jdk17u/pull/400/files
  - new: https://git.openjdk.org/jdk17u/pull/400/files/cdd40840..7b122c13

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk17u&pr=400&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk17u&pr=400&range=00-01

  Stats: 4 lines in 1 file changed: 0 ins; 4 del; 0 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