On Thu, 1 Jun 2023 20:25:16 GMT, Sean Mullan <mul...@openjdk.org> wrote:
>> Rajan Halade has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 8308592: Framework for CA interoperability testing > > test/jdk/security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java > line 47: > >> 45: * @throws Exception thrown when certificate can't be validated as >> valid or revoked >> 46: */ >> 47: public void validate(String caAlias, > > Seems like you could also add this as a static method to > `ValidatePathWithURL` and then you don't need this class. Would prefer to leave it as a separate class for better readability. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/14252#discussion_r1213671356
