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