8263442: Potential bug in jdk.internal.net.http.common.Utils.CONTEXT_RESTRICTED #2977
The fix for the reported bug in Utils.CONTEXT_RESTRICTED caused a couple of regression failures, which turned out to be another bug exposed by this fix where HTTP/1.1 CONNECT requests with authentication were filtering out proxy authentication headers wrongly. This was because the HttpRequestImpl created for the repeated CONNECT was putting the system headers in the user headers area of the HttpRequestImpl. The fix for that is to supply the user and system headers direct to the place where the new HttpRequestImpl is created.
To checkout this PR locally:
To update a local copy of the PR:
The text was updated successfully, but these errors were encountered:
@Michael-Mc-Mahon This change now passes all automated pre-integration checks.
After integration, the commit message for the final commit will be:
At the time when this comment was updated there had been 77 new commits pushed to the
As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details.
@Michael-Mc-Mahon Since your change was applied there have been 79 commits pushed to the
Your commit was automatically rebased without conflicts.
Pushed as commit bd7a184.