On Mon, 2 Oct 2023 13:17:16 GMT, Daniel Fuchs <[email protected]> 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
