[jdk21u-dev] RFR: 8335181: Incorrect handling of HTTP/2 GOAWAY frames in HttpClient [v3]
Alexey Bakhtin
abakhtin at openjdk.org
Wed Mar 19 17:55:31 UTC 2025
> Please review the backport of JDK-8335181
>
> src/java.net.http/share/classes/jdk/internal/net/http/MultiExchange.java
> - manual merge for MultiExchange::getExceptionalCF
> because of the difference in the logging caused by 8310330
>
> src/java.net.http/share/classes/jdk/internal/net/http/Http2Connection.java
> - added import for the AtomicLong class manually because of the difference in the import section
>
> src/java.net.http/share/classes/jdk/internal/net/http/ExchangeImpl.java
> - copyright year updated manually
>
> test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/http2/Http2TestServerConnection.java
> - import section is updated manually
>
> test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java
> - getConnectionKey() declaration is added manually in the HttpServerAdapters::HttpTestExchange class
>
> All jtreg tests for java.net.httpclient are passed
Alexey Bakhtin 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 four additional commits since the last revision:
- Merge branch 'master' into JDK-8335181
- Remove extra import
- Backport 720b44648bcff997278af92746f942b2425298a5
- Backport fc98998627443d6e73ac70661f47f48b30525712
-------------
Changes:
- all: https://git.openjdk.org/jdk21u-dev/pull/1417/files
- new: https://git.openjdk.org/jdk21u-dev/pull/1417/files/0ec8ff7f..bc13f76a
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk21u-dev&pr=1417&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk21u-dev&pr=1417&range=01-02
Stats: 31112 lines in 826 files changed: 21672 ins; 3706 del; 5734 mod
Patch: https://git.openjdk.org/jdk21u-dev/pull/1417.diff
Fetch: git fetch https://git.openjdk.org/jdk21u-dev.git pull/1417/head:pull/1417
PR: https://git.openjdk.org/jdk21u-dev/pull/1417
More information about the jdk-updates-dev
mailing list