I added a sentence about policy.expandProperties to the ConfigFile class
description. In the java.security file, I also clarified that setting
the policy.allowSystemProperty to false (in addition to commenting it
out) would disable the feature.
New webrev at http://cr.openjdk.java.net/~mullan/webrevs/8191395/webrev.01/
CSR: https://bugs.openjdk.java.net/browse/JDK-8238483
--Sean
On 1/31/20 10:42 AM, Sean Mullan wrote:
On 1/30/20 7:27 PM, Weijun Wang wrote:
Looks fine. Does this require a CSR?
Hmm, I was on the fence about that. ConfigFile already mentions
policy.allowSystemProperty in the class description [1], so I think in
this case, we are just improving the documentation.
I can't find any JAAS reference to policy.expandProperties though. I
suppose we should file one then since this would be the first time it is
documented.
Let me draft up a CSR and reply back later.
--Sean
[1]
https://download.java.net/java/early_access/jdk14/docs/api/jdk.security.auth/com/sun/security/auth/login/ConfigFile.html
Thanks,
Max
On Jan 31, 2020, at 3:18 AM, Sean Mullan <sean.mul...@oracle.com> wrote:
Please review this small change to the documentation of these two
security properties.
webrev: http://cr.openjdk.java.net/~mullan/webrevs/8191395/webrev.00/
bug: https://bugs.openjdk.java.net/browse/JDK-8191395
Thanks,
Sean
