RFR: 8314978: Multiple server call from connection failing with expect100 in getOutputStream [v3]

Vyom Tewari vtewari at openjdk.org
Wed Oct 4 10:45:43 UTC 2023


On Mon, 2 Oct 2023 13:17:16 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Vyom Tewari has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Incorporated the review comments
>
> test/jdk/java/net/HttpURLConnection/HttpURLConnectionExpect100Test.java line 82:
> 
>> 80:         conn.setRequestProperty("Expect", "100-continue");
>> 81:         sendRequest(conn);
>> 82:         getHeaderField(conn);
> 
> Could you call conn.getInputStream().readAllBytes() here to verify that it doesn't throw? Also verify that you get the expected 0 length result.

As getInputStream calls > getInputStream0 > getOutputStream > getOutputStream0 >  So if server rejected the operation  then  getInputStream will also throw the same exception. This behavior is same even without the fix.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15483#discussion_r1345587258


More information about the net-dev mailing list