On Fri, 29 Nov 2024 10:06:36 GMT, Michael McMahon <micha...@openjdk.org> wrote:
>> src/java.base/unix/classes/sun/net/www/protocol/http/ntlm/NTLMAuthentication.java
>> line 87:
>>
>>> 85: defaultDomain = props.getProperty("http.auth.ntlm.domain", "");
>>> 86: String ntlmCacheProp = props.getProperty("jdk.ntlm.cache",
>>> "true");
>>> 87: ntlmCache = Boolean.parseBoolean(ntlmCacheProp);
>>
>> Can't these simply be replaced with `System.getProperty()`? (There are more
>> usages of `privilegedGetProperties()` in this file, FYI.)
>
> Well spotted!
Thanks for noticing that!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22447#discussion_r1863277735
