RFR: 8339560: Unaddressed comments during code review of JDK-8337664
duke
duke at openjdk.org
Wed Sep 25 14:14:41 UTC 2024
On Wed, 11 Sep 2024 09:07:15 GMT, Fernando Guallini <fguallini at openjdk.org> wrote:
> During the code review of [JDK-8337664](https://bugs.openjdk.org/browse/JDK-8337664), several non-critical comments were raised but not addressed due to time constraints. This PR includes the following changes:
>
> 1. Symantec and Entrust tests now reuse common code. The Distrust.java class contains this reusable code, making it easier to add new tests for distrusted roots in the future.
> 2. In several instances where an Exception is thrown, it has been replaced with RuntimeException.
> 3. Many Symantec test certs have already expired. To properly test the distrusted scenario, this change wraps the intermediate and root CA certs in NonExpiringTLSServerCert, which does not validate expiration.
> 4. Package structure changed:
> <img width="225" alt="image" src="https://github.com/user-attachments/assets/a8c8407e-edd3-47dd-84c0-19feb9da3c0f">
@fguallini
Your change (at version 7e7a30f92ff952a6f9eeb4296f9103a3752801c0) is now ready to be sponsored by a Committer.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/20944#issuecomment-2374208089
More information about the security-dev
mailing list