On Tue, 2 May 2023 16:35:18 GMT, Jiangli Zhou <[email protected]> wrote:

> This PR was requested by [email protected]. The updates were provided by 
> [email protected].

> > > 
> > 
> > 
> > I have infra tests for interop implemented. @jianglizhou, please check 
> > https://github.com/openjdk/jdk/compare/master...rhalade:jdk:googletrust-certify?expand=1
> 
> @rhalade, thanks! I have a minor comment below for your 
> test/jdk/security/infra/java/security/cert/CertPathValidator/certification/GoogleCA.java
>  test. I'll defer to @[[email protected]](mailto:[email protected]) for 
> detailed review, as I don't have much context.
> 
> Please fix the bug id, `8303394`:
> 
> ```
> /*
>  * @test
>  * @bug 8303394
>  * @summary Interoperability tests with Google's GlobalSign R4 and GTS Root 
> certificates
>  ...
> ```
> 
> Could you please also let me know your plan on committing the GoogleCA.java? 
> Do you plan to create a PR?

You can include this contribution in your PR. Then it will be easier to 
backport to JDK 20u as one changeset. I updated bug id in the changeset.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/13754#issuecomment-1531972274

Reply via email to