RFR: 8292876: Do not include the deprecated userinfo component of the URI in HTTP/2 headers [v6]

Darragh Clarke duke at openjdk.org
Thu Oct 13 11:21:57 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 three additional commits since the last revision:

 - check authority pseudoheader directly
 - removed unused import
 - test cleanup

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/10592/files
  - new: https://git.openjdk.org/jdk/pull/10592/files/67a3f84b..d2d48054

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

  Stats: 45 lines in 1 file changed: 22 ins; 18 del; 5 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