Request for Review 6882594

Christopher Hegarty -Sun Microsystems Ireland Christopher.Hegarty at Sun.COM
Thu Sep 17 03:55:07 PDT 2009


I removed the use of char (schemeId) completed and replaced it the the 
new AuthScheme enum. This makes the code much cleaner. Thanks Max and 
Alan for this suggestion.

Updated webrev can be found at:
   http://cr.openjdk.java.net/~chegar/6882594/webrev.1/webrev/

Thanks,
-Chris.

Christopher Hegarty - Sun Microsystems Ireland wrote:
> Hi Jessie, Michael, Alan,
> 
> 6882594: Remove static dependency on NTLM authentication
> 
> Webrev:
>   http://cr.openjdk.java.net/~chegar/6882594/webrev.0
> 
> The HTTP client implementation 
> (sun.net.www.protocol.http.HttpURLConnection) loads the NTLM 
> Authentication classes even if no authentication is used. These classes 
> should not be loaded unless required. Also, make NTLM a runtime 
> dependency rather than a static one.
> 
> I took the liberty of cleaning up some imports along the way.
> 
> -Chris.



More information about the net-dev mailing list