[jdk17u-dev] RFR: 8306040: HttpResponseInputStream.available() returns 1 on empty stream

Goetz Lindenmaier goetz at openjdk.org
Wed Mar 20 13:31:22 UTC 2024


On Wed, 6 Mar 2024 14:23:34 GMT, Sonia Zaldana Calles <szaldana at openjdk.org> wrote:

>> Hi all, 
>> 
>> This pull request contains a backport of commit [acaab6fd74f507bb6b18167505d88e505bdf24bd](https://github.com/openjdk/jdk/commit/acaab6fd74f507bb6b18167505d88e505bdf24bd) from the [openjdk/jdk](https://github.com/openjdk/jdk) repository. 
>> 
>> Testing: using the reproducer linked in the issue, I was able to verify the patch. 
>> I also ran the additional test case included in the original commit and all tests in ```test/jdk/java/net/httpclient/```. 
>> 
>> Note that patch is not clean because http tests and related classes have since been moved into a package of their own (namely ```/test/jdk/java/net/httpclient/lib```), which doesn’t exist in jdk 17. 
>> 
>> Only change introduced in my patch from the original commit: 
>> 
>> @@ -25,7 +25,7 @@
>>   * @test
>>   * @bug 8306040
>>   * @summary HttpResponseInputStream.available() returns 1 on empty stream
>> - * @library /test/lib /test/jdk/java/net/httpclient/lib
>> + * @library /test/lib
>>   * @run junit/othervm HttpInputStreamAvailableTest
>>   *
>>   */
>
> GHA failure: Unrelated Risk-V build issue.

Hi @SoniaZaldana, what about this change? It is pending for a long time now, and it should go to the July release please.

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

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/2263#issuecomment-2009566901


More information about the jdk-updates-dev mailing list