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 <[email protected]>
> Sent: Donnerstag, 30. Januar 2020 09:25
> To: Sean Mullan <[email protected]>; security-
> [email protected]
> Cc: Langer, Christoph <[email protected]>
> 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