Re: RFR 8171319: keytool should print out warnings when reading or generating cert/cert req using weak algorithms

2017-02-15 Thread Seán Coffey
Hi Weijun, That's looks good to me and will be a big help for keytool usability. some thoughts : Main.java : in your printCRL method, would you consider editing the X509CRLImpl class to print with a customized string ? It'll make the code more resilient to future changes in this area i.e. s

Re: RFR 8168410: Multiple JCK tests are failing due to SecurityException is not thrown.

2017-02-15 Thread Weijun Wang
The last existing paragraph is already A system property named jdk.io.permissionsUseCanonicalPath has also been introduced. When it is set to "true", FilePermission will canonicalize its pathname as it did before JDK 9. The default value of this property is "false". and I'd like the first

Re: RFR 8168410: Multiple JCK tests are failing due to SecurityException is not thrown.

2017-02-15 Thread Sean Mullan
On 2/15/17 3:35 AM, Weijun Wang wrote: I's also like to append the following paragraph to the current release note at https://bugs.openjdk.java.net/browse/JDK-8165836: Another new system property named jdk.security.filePermCompat, when set to "true", allows the compatibility layer above to wo

Re: RFR 8168410: Multiple JCK tests are failing due to SecurityException is not thrown.

2017-02-15 Thread Sean Mullan
Looks good. --Sean On 2/14/17 8:04 PM, Weijun Wang wrote: Webrev updated at http://cr.openjdk.java.net/~weijun/8168410/webrev.01/ Change since webrev.00: http://cr.openjdk.java.net/~weijun/8168410/webrev.01/interdiff.patch.html Thanks Max On 02/14/2017 11:55 PM, Sean Mullan wrote: Hi

Re: RFR 8168410: Multiple JCK tests are failing due to SecurityException is not thrown.

2017-02-15 Thread Weijun Wang
I's also like to append the following paragraph to the current release note at https://bugs.openjdk.java.net/browse/JDK-8165836: Another new system property named jdk.security.filePermCompat, when set to "true", allows the compatibility layer above to work on third-party Policy implementati