Hi Matthias,

I'm wondering whether we should add cpve as cause of the RuntimeException in 
test/jdk/security/infra/java/security/cert/CertPathValidator/certification/ValidatePathWithParams.java,
 line 177, instead of printing it out explicitly?

Like:
throw new RuntimeException(
                        "TEST FAILED: couldn't determine EE certificate 
status", cvpe);

Best regards
Christoph

> -----Original Message-----
> From: Baesken, Matthias <matthias.baes...@sap.com>
> Sent: Donnerstag, 30. Januar 2020 09:25
> To: Sean Mullan <sean.mul...@oracle.com>; security-
> d...@openjdk.java.net
> Cc: Langer, Christoph <christoph.lan...@sap.com>
> Subject: RE: RFR: 8237962: give better error output for invalid OCSP response
> intervals in CertPathValidator checks
> 
> Thanks, may I have a second review ?
> 
> Best regards, Matthias
> 
> 
> 
> > Looks good.
> >
> > Thanks,
> > Sean
> >
> > On 1/29/20 4:20 AM, Baesken, Matthias wrote:
> > >
> > > Hi Sean, new webrev :
> > >
> > >
> > > http://cr.openjdk.java.net/~mbaesken/webrevs/8237962.1/
> > >
> > >
> > > Best Regards, Matthias
> > >

Reply via email to