[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