RFR: [jdk11u]: 8211437 java.net.http.HttpClient hangs on 204 reply without Content-length 0

[Daniel Fuchs]

Hi Michael,

The diff compared to the original changeset looks good to me.
I assume you have double checked that jtreg was happy with it!

Reviewed.

best regards,

-- daniel

On 24/10/2018 11:03, Michael McMahon wrote:
> Hi,
> 
> This has already been approved assuming that the jdk changeset applied 
> cleanly.
> A small test change was made to jdk since approval was sought so the 
> changeset
> no longer applies cleanly. The full webrev is at:
> 
> http://cr.openjdk.java.net/~michaelm/8211437.jdk11u/webrev.1/
> 
> Thanks,
> Michael
> 
> The following is the patch for jdk11u relative to jdk
> 
> diff -r 6bc6fc7c8bb5 test/jdk/java/net/httpclient/http2/NoBodyTest.java
> --- a/test/jdk/java/net/httpclient/http2/NoBodyTest.java        Fri Oct 
> 19 14:23:43 2018 +0100
> +++ b/test/jdk/java/net/httpclient/http2/NoBodyTest.java        Wed Oct 
> 24 10:51:52 2018 +0100
> @@ -24,8 +24,8 @@
>   /*
>    * @test
>    * @bug 8087112
> - * @library /test/lib server
> - * @build jdk.test.lib.net.SimpleSSLContext
> + * @library /lib/testlibrary/ server
> + * @build jdk.testlibrary.SimpleSSLContext
>    * @modules java.base/sun.net.www.http
>    *          java.net.http/jdk.internal.net.http.common
>    *          java.net.http/jdk.internal.net.http.frame
> @@ -45,7 +45,7 @@
>   import java.net.http.HttpResponse;
>   import java.net.http.HttpResponse.BodyHandlers;
>   import java.util.concurrent.*;
> -import jdk.test.lib.net.SimpleSSLContext;
> +import jdk.testlibrary.SimpleSSLContext;
>   import org.testng.annotations.Test;
>   import static java.net.http.HttpClient.Version.HTTP_2;
> 
> 
>