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

Darragh Clarke duke at openjdk.org
Tue Oct 11 11:31:37 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 test

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

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

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

  Stats: 9 lines in 1 file changed: 5 ins; 2 del; 2 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