RFR: 8170305: URLConnection doesn't handle HTTP/1.1 1xx (informational) messages [v5]

Jaikiran Pai jpai at openjdk.org
Wed Sep 14 00:56:53 UTC 2022


On Mon, 12 Sep 2022 13:26:45 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Can I please get a review of this change which proposes to fix https://bugs.openjdk.org/browse/JDK-8170305?
>> 
>> The commit in this PR changes the internal implementation of `HttpURLConnection` to ignore interim informational 1xx responses from server and continue to wait for the final response. This is a similar fix to what we did in the new `HttpClient` API in https://github.com/openjdk/jdk/pull/10169.
>> 
>> A new test has been added to use the `HttpURLConnection` to reproduce the issue and verify the fix.
>> 
>> tier1, tier2 and tier3 testing is in progress with this change.
>
> Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:
> 
>   verify ProtocolException in the test

Hello Julian,

> Is this something that would be a candidate for backporting?

This change is relatively straightforward and backporting should be OK. OpenJDK backports however are decided/managed by different team(s) and discussed in https://mail.openjdk.org/mailman/listinfo/jdk-updates-dev mailing list. The OpenJDK guide has more details on the backporting process https://openjdk.org/guide/#backporting

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

PR: https://git.openjdk.org/jdk/pull/10229


More information about the net-dev mailing list