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

[Michael McMahon]

Thanks Daniel.

Yes, jtreg is happy with it :-)

Michael.

On 24/10/2018, 11:13, Daniel Fuchs wrote:
> 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;
>>
>>
>>
>