[jdk8u-dev] RFR: 8350498: Remove two Camerfirma root CA certificates [v2]
Antonio Vieiro
duke at openjdk.org
Mon May 19 08:30:43 UTC 2025
On Wed, 14 May 2025 18:14:42 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:
>> Antonio Vieiro has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Missing changes in commit
>
> jdk/test/sun/security/lib/cacerts/VerifyCACerts.java line 1:
>
>> 1: /*
>
> This file is missing the bug line update that the JDK 11u patch has here:
> https://github.com/openjdk/jdk11u-dev/commit/8894d6ad1b83553a5c60af51fd2de2319fd1d6ec#diff-ecedb2ba9c75491cd95fc614c6580b565d2d61a10a253f6a2fcd0fd774b535c3R31
Thanks for noticing! Uploaded in the recent commit.
> jdk/test/sun/security/ssl/X509TrustManagerImpl/distrust/Camerfirma.java line 37:
>
>> 35: * @summary Check that TLS Server certificates chaining back to distrusted
>> 36: * Camerfirma root are invalid
>> 37: * @library /test/lib
>
> The `roots` => `root` change is fine. The test library change not. 8u has `lib/security` not `/test/lib`.
Yep, uploaded the changes in the recent commit. Sorry by that.
-------------
PR Review Comment: https://git.openjdk.org/jdk8u-dev/pull/651#discussion_r2095140241
PR Review Comment: https://git.openjdk.org/jdk8u-dev/pull/651#discussion_r2095139438
More information about the jdk8u-dev
mailing list