> Hi, > > I closed https://github.com/openjdk/jdk/pull/21249 and am continuing the > review on this PR. > > This fix relaxes the constraints on user set authentication headers. > Currently, any user set authentication headers are filtered out, if the > HttpClient has an Authenticator set. The reason being that the authenticator > is expected to manage authentication. With this fix, it will be possible to > use pre-emptive authentication through user set headers, even if an > authenticator is set. The expected use case is where the authenticator would > manage either proxy or server authentication and the user set headers would > manage server authentication if the authenticator is managing proxy (or vice > versa). > > A CSR will be filed to document this change. > > Thanks, > Michael
Michael McMahon has updated the pull request incrementally with one additional commit since the last revision: Jai review ------------- Changes: - all: https://git.openjdk.org/jdk/pull/21408/files - new: https://git.openjdk.org/jdk/pull/21408/files/90419e45..2d9a4308 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=21408&range=03 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=21408&range=02-03 Stats: 8 lines in 4 files changed: 1 ins; 1 del; 6 mod Patch: https://git.openjdk.org/jdk/pull/21408.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/21408/head:pull/21408 PR: https://git.openjdk.org/jdk/pull/21408