From d85ea3134532d90297fdd94977fcff61be601d5b Mon Sep 17 00:00:00 2001 From: Andreas Kohn Date: Fri, 23 Sep 2022 13:19:40 +0200 Subject: [PATCH] Avoid nil dereferencing on errors This merges #204 to a similar place in config.go. --- config.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/config.go b/config.go index 99b47193..d9381fd9 100644 --- a/config.go +++ b/config.go @@ -785,7 +785,9 @@ func (cfg *Config) renewCert(ctx context.Context, name string, force, interactiv // try to obtain from each issuer until we succeed var issuedCert *IssuedCertificate var issuerUsed Issuer + var issuerKeys []string for _, issuer := range cfg.Issuers { + issuerKeys = append(issuerKeys, issuer.IssuerKey()) if prechecker, ok := issuer.(PreChecker); ok { err = prechecker.PreCheck(ctx, []string{name}, interactive) if err != nil { @@ -818,7 +820,7 @@ func (cfg *Config) renewCert(ctx context.Context, name string, force, interactiv "renewal": true, "identifier": name, "remaining": timeLeft, - "issuer": issuerUsed.IssuerKey(), + "issuers": issuerKeys, "storage_key": certRes.NamesKey(), "error": err, })