Re: RFR: 8198240: Allow cacerts test to pass when GTECyberTrust root expires

2018-04-13 Thread Sean Mullan
On 4/13/18 11:46 AM, Rajan Halade wrote: Thanks for your comments. Updated webrev is at http://cr.openjdk.java.net/~rhalade/8198240/webrev.01/ Looks fine. --Sean

Re: RFR: 8198240: Allow cacerts test to pass when GTECyberTrust root expires

2018-04-13 Thread Rajan Halade
Thanks for your comments. Updated webrev is at http://cr.openjdk.java.net/~rhalade/8198240/webrev.01/ - Rajan On 4/13/18 5:59 AM, Sean Mullan wrote: 288 System.err.println("WARNING: "); I don't think you need to print a warning in this case since this expired root is an exception to the poli

Re: RFR: 8198240: Allow cacerts test to pass when GTECyberTrust root expires

2018-04-13 Thread Sean Mullan
288 System.err.println("WARNING: "); I don't think you need to print a warning in this case since this expired root is an exception to the policy. Also, once the cert has expired, the subsequent message "will expire" doesn't make sense: 293 System.err.prin

RFR: 8198240: Allow cacerts test to pass when GTECyberTrust root expires

2018-04-12 Thread Rajan Halade
Please review this test fix to allow exception list to 90 days expiry policy. Fix also modifies the error reporting to report alias of certificate being checked. Webrev: http://cr.openjdk.java.net/~rhalade/8198240/webrev.00/ Thanks, Rajan