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

Jaikiran Pai jpai at openjdk.org
Mon Sep 12 13:26:45 UTC 2022


> 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

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/10229/files
  - new: https://git.openjdk.org/jdk/pull/10229/files/c4b62c5b..cb9364ac

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=10229&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10229&range=03-04

  Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/10229.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/10229/head:pull/10229

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


More information about the net-dev mailing list