[jdk21u-dev] RFR: 8335181: Incorrect handling of HTTP/2 GOAWAY frames in HttpClient [v2]

[Andrew John Hughes]

On Thu, 20 Feb 2025 23:46:34 GMT, Alexey Bakhtin <abakhtin at openjdk.org> wrote:

>> 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 incrementally with one additional commit since the last revision:
> 
>   Remove extra import

Can you rebase this onto current jdk21u-dev? At the moment I'm seeing code from 8316580 in the diff? Thanks.

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

PR Comment: https://git.openjdk.org/jdk21u-dev/pull/1417#issuecomment-2737424982