Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8339560

Unaddressed comments during code review of JDK-8337664

XMLWordPrintable

    • 24
    • b18

        During the review, Rajan and Sean had several comments that I was unable to address because of time constraints. Nothing was urgent so it was decided to file a follow-on bug.

        1. test/jdk/sun/security/ssl/X509TrustManagerImpl/Entrust/Distrust.java, line 113:
        Rajan - "Please update this and other Exception thrown in loadCertificateChain and testTM function to RuntimeException."

        2. test/jdk/sun/security/ssl/X509TrustManagerImpl/Entrust/Distrust.java, line 141:
        Rajan - "Should this be updated to throw SkippedException so we know that certificates are expired?"

        3. Sean - "I noticed we could combine some of the Symantec and Entrust tests, but I think it is too risky at this point."

              fguallini Fernando Guallini
              mpowers Mark Powers
              Votes:
              0 Vote for this issue
              Watchers:
              10 Start watching this issue

                Created:
                Updated:
                Resolved: