On Wed, 11 Sep 2024 09:07:15 GMT, Fernando Guallini <fguall...@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";> This pull request has now been integrated. Changeset: 95d3e9d1 Author: Fernando Guallini <fguall...@openjdk.org> Committer: Sean Mullan <mul...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/95d3e9d199600bac0284f9151b99aef152e027ac Stats: 932 lines in 26 files changed: 391 ins; 541 del; 0 mod 8339560: Unaddressed comments during code review of JDK-8337664 Reviewed-by: mullan ------------- PR: https://git.openjdk.org/jdk/pull/20944
