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