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 policy. Also, once the cert has
expired, the subsequent message "will expire" doesn't make sense:
293 System.err.println("cert \"" + alias + "\" expiry
\""
294 + notAfter.toString() + "\" will expire
within 90 days");
since it has already expired. Just print out the message above for the
error cases.
Looks good otherwise.
--Sean
On 4/12/18 5:40 PM, Rajan Halade wrote:
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
