Re: RFR: 8160655 Fix denyAfter and usage types for security properties

2017-02-06 Thread Sean Mullan
Hi Tony, Here are my comments on the latest webrev: * SignerInfo.java 355 try { 356 JAR_DISABLED_CHECK.permits(digestAlgname, cparams); 357 } catch (CertPathValidatorException e) { 358 throw new SignatureException(e.g

RFR 8173410: Add commented config line for jdk.security.provider.preferred

2017-02-06 Thread Anthony Scarpino
Please review this change for Solaris SPARC configurations to add an optional configuration line. There are a few minor changes which didn't change any content. Changing some "#"'s for a bit more consistency and readability across the file. Also moving an RMI entry to the end that was merged

Re: [9] 8173956: KeyStore regression due to default keystore being changed to PKCS12

2017-02-06 Thread Xuelei Fan
Looks fine to me. Xuelei On 2/6/2017 8:29 AM, Vincent Ryan wrote: Please review this fix to correct support for mixed-case aliases in PKCS12 keystores. Thanks. Webrev: http://cr.openjdk.java.net/~vinnie/8173956/webrev.00/ Bug: https://bugs.openjdk.java.net/browse/JDK-8173956

[9] 8173956: KeyStore regression due to default keystore being changed to PKCS12

2017-02-06 Thread Vincent Ryan
Please review this fix to correct support for mixed-case aliases in PKCS12 keystores. Thanks. Webrev: http://cr.openjdk.java.net/~vinnie/8173956/webrev.00/ Bug: https://bugs.openjdk.java.net/browse/JDK-8173956