On 20 Apr 2016, at 20:34, Claes Redestad <claes.redes...@oracle.com> wrote:
> > > On 2016-04-20 20:51, Chris Hegarty wrote: >> On 20 Apr 2016, at 15:44, Claes Redestad <claes.redes...@oracle.com> wrote: >> >>> Hello, >>> >>> now that the sun.security.action package is encapsulated we can simplify >>> internal code to get System properties. >>> >>> Bug: https://bugs.openjdk.java.net/browse/JDK-8154231 >>> Webrev: http://cr.openjdk.java.net/~redestad/8154231/webrev.01/ >> This looks very nice. > > Thanks! > >> >> Did you accidentally remove supportsTransparentAuth in the unix version >> of NTLMAuthentication.java ? It is used reflectively by >> sun.net.www.protocol.http.NTLMAuthenticationProxy. Otherwise, this looks >> fine to me. > > Oops, nice catch. Updated. I’m happy with this version. Thanks. -Chris.