fix: update

Signed-off-by: Junjie Gao <[email protected]>
notaryproject · Aug 22, 2024 · c620d85 · c620d85
1 parent d643ea2
commit c620d85
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/revocation/internal/crl/crl.go b/revocation/internal/crl/crl.go
@@ -69,9 +69,9 @@ func CertCheckStatus(ctx context.Context, cert, issuer *x509.Certificate, opts C
 		// CRL not enabled for this certificate.
 		return &result.CertRevocationResult{
 			Result: result.ResultNonRevokable,
-			CRLResults: []*result.CRLResult{
-				{Result: result.ResultNonRevokable},
-			},
+			CRLResults: []*result.CRLResult{{
+				Result: result.ResultNonRevokable,
+			}},
 		}
 	}
 

diff --git a/revocation/internal/crl/crl_test.go b/revocation/internal/crl/crl_test.go
@@ -33,6 +33,14 @@ import (
 )
 
 func TestCertCheckStatus(t *testing.T) {
+	t.Run("certtificate does not have CRLDistributionPoints", func(t *testing.T) {
+		cert := &x509.Certificate{}
+		r := CertCheckStatus(context.Background(), cert, &x509.Certificate{}, CertCheckStatusOptions{})
+		if r.Result != result.ResultNonRevokable {
+			t.Fatalf("expected NonRevokable, got %s", r.Result)
+		}
+	})
+
 	t.Run("download error", func(t *testing.T) {
 		cert := &x509.Certificate{
 			CRLDistributionPoints: []string{"http://example.com"},