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