[jdk11u-dev] RFR: 8316138: Add GlobalSign 2 TLS root certificates [v2]

Fri May 17 14:32:22 UTC 2024

On Fri, 17 May 2024 14:22:35 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

>> María Arias de Reyna has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix switch expression that is too new for this JDK
>
> test/jdk/security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java line 659:
> 
>> 657:             case "globalsignr46" ->
>> 658:                     new CATestURLs("https://valid.r46.roots.globalsign.com",
>> 659:                             "https://revoked.r46.roots.globalsign.com");
> 
> Does this compile? Switch expressions are JDK 14+.

It did compile for me... maybe I compiled it with the wrong JDK!

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

PR Review Comment: https://git.openjdk.org/jdk11u-dev/pull/2715#discussion_r1605101082