[jdk17u-dev] Withdrawn: 8342075: HttpClient: improve HTTP/2 flow control checks

[duke]

On Mon, 24 Feb 2025 23:37:26 GMT, Alexey Bakhtin <abakhtin at openjdk.org> wrote:

> Please review backport of JDK-8342075
> 
> The backport is not clean:
> 
> - src/java.net.http/share/classes/module-info.java
> 
> the changes are skipped because there are no java.net.http configuration properties definitions in the file. It was added by JDK-8296804 in JDK20 but not backported.
> 
> - src/java.net.http/share/classes/jdk/internal/net/http/Stream.java
> 
> Comments are not added into Stream::incoming() method because of no JDK-8298340 in JDK17
> 
> - src/java.net.http/share/classes/jdk/internal/net/http/WindowUpdateSender.java
> 
> manual merge because of differences in the import section caused by JDK-8308310
> 
> - src/java.net.http/share/classes/jdk/internal/net/http/Http2Connection.java
> 
> Add Http2Connection.java::isOpen method. It was added by 8254786 in JDK19
> 
> - test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/http2/Http2TestExchange.java  and test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/http2/Http2TestExchangeImpl.java and test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/http2/Http2TestServerConnection.java
> 
> Test classes are updated from JDK-8335181 with getConnectionKey() implementation
> 
> -  test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/http2/ConnectionFlowControlTest.java and test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/http2/StreamFlowControlTest.java
> 
> Test classes are updated to use ExecutorService::shutdown method to shut down httpclient. These methods are added by JDK-8267140 in JDK21
> 
> java/net/httpclient jtreg tests are passed

This pull request has been closed without being integrated.

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

PR: https://git.openjdk.org/jdk17u-dev/pull/3298