fix: update

Signed-off-by: Junjie Gao <[email protected]>
notaryproject · Sep 14, 2024 · deec55e · deec55e
1 parent 50f6e6d
commit deec55e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/revocation/internal/crl/crl.go b/revocation/internal/crl/crl.go
@@ -92,7 +92,7 @@ func CertCheckStatus(ctx context.Context, cert, issuer *x509.Certificate, opts C
 		lastErr       error
 		crlURL        string
 	)
-	for _, crlURL := range cert.CRLDistributionPoints {
+	for _, crlURL = range cert.CRLDistributionPoints {
 		baseCRL, err := download(ctx, crlURL, opts.HTTPClient)
 		if err != nil {
 			lastErr = fmt.Errorf("failed to download CRL from %s: %w", crlURL, err)

diff --git a/revocation/result/results.go b/revocation/result/results.go
@@ -77,7 +77,7 @@ type ServerResult struct {
 	Error error
 
 	// RevocationMethod is the method used to check the revocation status of the
-	// certificate, including Unknown(0), MethodOCSP(1), MethodCRL(2)
+	// certificate, including MethodUnknown, MethodOCSP, MethodCRL
 	RevocationMethod revocation.Method
 }
 
@@ -119,7 +119,7 @@ type CertRevocationResult struct {
 	ServerResults []*ServerResult
 
 	// RevocationMethod is the method used to check the revocation status of the
-	// certificate, including Unknown(0), MethodOCSP(1), MethodCRL(2) and
-	// MethodOCSPFallbackCRL(3)
+	// certificate, including MethodUnknown, MethodOCSP, MethodCRL and
+	// MethodOCSPFallbackCRL
 	RevocationMethod revocation.Method
 }