On Thu, 13 Oct 2022 11:21:57 GMT, Darragh Clarke <[email protected]> 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 > > 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 Marked as reviewed by jpai (Reviewer). Thank you Darragh for those changes. The latest state of this PR in `d2d48054` looks good to me. ------------- PR: https://git.openjdk.org/jdk/pull/10592
