On Mon, 5 Aug 2024 19:01:30 GMT, Jamil Nimeh <jni...@openjdk.org> wrote:

> This proposed enhancement adds logging to the OCSPTimeout test, which is 
> intermittently failing and difficult to reproduce.  The hope is that with 
> extra logging enabled that additional clues as to the cause of these rare 
> failures will become apparent.
> 
> JBS: https://bugs.openjdk.org/browse/JDK-8337826

Changes requested by mullan (Reviewer).

test/jdk/java/security/cert/CertPathValidator/OCSP/OCSPTimeout.java line 32:

> 30: /**
> 31:  * @test
> 32:  * @bug 8179502 8337826

You should only include the bugid if the test is testing a specific bug fix. In 
this case, it is just additional debugging so no need to add it.

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

PR Review: https://git.openjdk.org/jdk/pull/20471#pullrequestreview-2221160888
PR Review Comment: https://git.openjdk.org/jdk/pull/20471#discussion_r1705449501

Reply via email to