RFR: 8352502: Response message is null if expect 100 assertion fails with non 100 [v2]
Darragh Clarke
dclarke at openjdk.org
Tue Jul 8 14:33:04 UTC 2025
> Currently if a request has set Expect-Continue and receives a non 100 response the `responseMessage` wouldn't be set.
>
> This PR sets `responseMessage`, it also updates `getResponseMessage` to check if the message has already been set. This should match the way that `responseCode` is currently handled.
>
> I also added a test to cover some possible responses.
Darragh Clarke has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
- removed unneeded library
- more cleanup
- moved test, implemented feedback
- merged master into branch
- implementing comments
- Response Message could be null
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/25999/files
- new: https://git.openjdk.org/jdk/pull/25999/files/10563157..d6f27780
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=25999&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=25999&range=00-01
Stats: 113948 lines in 2239 files changed: 68346 ins; 28980 del; 16622 mod
Patch: https://git.openjdk.org/jdk/pull/25999.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/25999/head:pull/25999
PR: https://git.openjdk.org/jdk/pull/25999
More information about the net-dev
mailing list