bug fix:OCSP missing values (#138)

projectdiscovery · Dec 15, 2022 · c9831d8 · c9831d8
1 parent 477cee6
commit c9831d8
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/pkg/tlsx/clients/clients.go b/pkg/tlsx/clients/clients.go
@@ -324,7 +324,12 @@ func IsTLSRevoked(cert *x509.Certificate) bool {
 
 // IsZTLSRevoked returns true if the certificate has been revoked
 func IsZTLSRevoked(cert *zx509.Certificate) bool {
-	OCSPisRevoked, _, OCSPerr := zverifier.CheckOCSP(context.TODO(), cert, nil)
+	var OCSPisRevoked bool = false
+	var OCSPerr error
+	// TODO : Verify Upstream Patch and remove extra condition when fixed
+	if len(cert.IssuingCertificateURL) > 0 && len(cert.OCSPServer) > 0 {
+		OCSPisRevoked, _, OCSPerr = zverifier.CheckOCSP(context.TODO(), cert, nil)
+	}
 	if len(cert.CRLDistributionPoints) != 0 {
 		CRLisRevoked, _, CRLerr := zverifier.CheckCRL(context.TODO(), cert, nil)