On Wed, 25 Aug 2021 14:21:59 GMT, Alex Kasko <aka...@openjdk.org> wrote:

>> When HEAD request is used with a proxy (or a server) that requires NTLM, 
>> authentication fails when server returns large (8kb+) body along with 
>> NTLMSSP_CHALLENGE response.
>> 
>> Proposed fix is to check for ongoing NTLM auth in `reset()` and consume the 
>> response body in this case.
>> 
>> Alternatively the whole check for `HEAD` method in `reset()` can be dropped.
>
> Alex Kasko has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   fix direct server and plain http proxy auth that became inadvertently broken

Could you please fix the PR? When I click on "changes" I see a one-line fix in 
HttpURLConnection which involves a new `TunnelState` enum which I have never 
seen before. It looks as if something is wrong with the PR - it's not showing 
all the changes?

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

PR: https://git.openjdk.java.net/jdk/pull/4753

Reply via email to