Hi Chris
We just had a 1.5-day team building and a weekend, and I just noticed
this mail.
The disconnectWeb() method was added in "6586707: NTLM authentication
with proxy fails" to support 2-layer NTLM authentications. There was no
regression test to that fix. I'll need a little time to setup an
environment to check if your new code change has any effect on it.
Thanks
Max
On 06/23/2010 10:08 PM, Chris Hegarty wrote:
Hi Michael, Max,
This is a code review request for CR 6954525: Testcase failure
java/net/Authenticator/B4769350.java.
Essentially, there is a race among requesting threads in the test. The
threads are making HTTP requests requiring authentication. This is fine,
and what the test is trying to achieve, but some of the final replies
from the server/proxy closed the connection after they send the response
without notifying the HTTP client. If these connections go back into the
keep alive cache they may cause problems when/if reused by another
thread. See bug eval section for more details.
The solution is to include the 'Connection: close' header if the
server/proxy is going to close the connection.
Also, sun.net.www.protocol.http.HttpURLConnection.disconnectWeb should
check if the connection is to be kept alive before automatically resetting.
Webrev:
http://cr.openjdk.java.net/~chegar/6954525/webrev.00/webrev/
Thanks,
-Chris.