RFR: 8292876: Do not include the deprecated userinfo component of the URI in HTTP/2 headers [v2]
Darragh Clarke
duke at openjdk.org
Fri Oct 7 11:18:15 UTC 2022
> Changed the way the `:authority` pseudo header is set to only include host and, if available, port.
> I added a test to cover this change that consists of a HttpClient that makes a request which contains userInfo, the test passes if the request is carried out with the userInfo not being added to the `:authority` header.
>
>
> ### Tests
> I ran Tier 1 - Tier 3 tests, as well as paying special attention to the http client tests to make sure they consistently passed
Darragh Clarke has updated the pull request incrementally with one additional commit since the last revision:
fixed formatting, added finally to cleanup server if test fails
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/10592/files
- new: https://git.openjdk.org/jdk/pull/10592/files/dec42cb2..7a9c9d04
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=10592&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=10592&range=00-01
Stats: 15 lines in 2 files changed: 4 ins; 1 del; 10 mod
Patch: https://git.openjdk.org/jdk/pull/10592.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/10592/head:pull/10592
PR: https://git.openjdk.org/jdk/pull/10592
More information about the net-dev
mailing list