Integrated: 8292876: Do not include the deprecated userinfo component of the URI in HTTP/2 headers

Darragh Clarke duke at openjdk.org
Fri Oct 14 01:40:10 UTC 2022


On Thu, 6 Oct 2022 11:35:32 GMT, Darragh Clarke <duke at openjdk.org> wrote:

> 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

This pull request has now been integrated.

Changeset: b30d9220
Author:    Darragh Clarke <Darragh.clarke at oracle.com>
Committer: Jaikiran Pai <jpai at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/b30d922009bab114fb8ac200df42b594a72a2942
Stats:     123 lines in 2 files changed: 120 ins; 0 del; 3 mod

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

Reviewed-by: aefimov, dfuchs, jpai

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

PR: https://git.openjdk.org/jdk/pull/10592


More information about the net-dev mailing list