RFR: 8054022: HttpURLConnection timeouts with Expect: 100-Continue and no chunking [v4]
Darragh Clarke
duke at openjdk.org
Wed May 3 11:47:13 UTC 2023
> Currently it is possible for `HttpURLConnection` with the `Expect: 100-Continue` header to timeout awaiting for a server response. According to [RFC-7231](https://www.rfc-editor.org/rfc/rfc7231#section-5.1.1) a client `SHOULD NOT wait for an indefinite period before sending the message body`.
>
> This PR changes the existing `expect100Continue` method to wait for a maximum of 5 seconds for a server response, this will be shorter if a timeout is set. If no response is received, the message is sent regardless.
>
> Tests have been added to account for different scenarios that currently timeout, and the changes have been tested against tiers 1,2 and 3.
Darragh Clarke has updated the pull request incrementally with one additional commit since the last revision:
Apply suggestions from code review
Co-authored-by: Daniel Fuchs <67001856+dfuch at users.noreply.github.com>
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/13330/files
- new: https://git.openjdk.org/jdk/pull/13330/files/9483e957..874e3497
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=13330&range=03
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=13330&range=02-03
Stats: 12 lines in 1 file changed: 10 ins; 0 del; 2 mod
Patch: https://git.openjdk.org/jdk/pull/13330.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/13330/head:pull/13330
PR: https://git.openjdk.org/jdk/pull/13330
More information about the net-dev
mailing list