From d84a81e12d38d0269fa019cdd5a805848ff6b36d Mon Sep 17 00:00:00 2001
From: Ainar Garipov <ainar-g@users.noreply.github.com>
Date: Thu, 13 Apr 2023 16:24:15 +0300
Subject: [PATCH] all: fix another panic in cert parsing (#21)

Fixes #20.
---
 client.go | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/client.go b/client.go
index 6f3e19d..5ad7572 100644
--- a/client.go
+++ b/client.go
@@ -79,7 +79,9 @@ func (c *Client) DialStamp(stamp dnsstamps.ServerStamp) (*ResolverInfo, error) {
 	if err != nil {
 		return nil, err
 	}
+
 	resolverInfo.SharedKey = sharedKey
+
 	return resolverInfo, nil
 }
 
@@ -251,7 +253,10 @@ func (c *Client) fetchCert(stamp dnsstamps.ServerStamp) (cert *Cert, err error)
 		cert, err = parseCert(stamp, currentCert, providerName, strings.Join(txt.Txt, ""))
 		if err != nil {
 			log.Debug("[%s] bad cert: %s", providerName, err)
+
+			continue
 		} else if cert == nil {
+			// The certificate has been skipped due to Serial or EsVersion.
 			continue
 		}
 
@@ -262,7 +267,7 @@ func (c *Client) fetchCert(stamp dnsstamps.ServerStamp) (cert *Cert, err error)
 	if foundValid {
 		return currentCert, nil
 	} else if err == nil {
-		err = errors.Error("no valid txt records")
+		err = fmt.Errorf("no valid txt records for provider %q", providerName)
 	}
 
 	return nil, err