chore: minor changes (#2108)

.golangci.yml

@@ -7,8 +7,6 @@ linters-settings:
     check-shadowing: true
   gocyclo:
     min-complexity: 12
-  maligned:
-    suggest-new: true
   goconst:
     min-len: 3
     min-occurrences: 3

certcrypto/crypto_test.go

@@ -39,14 +39,14 @@ func TestGenerateCSR(t *testing.T) {
 		expected   expected
 	}{
 		{
-			desc:       "without SAN",
+			desc:       "without SAN (nil)",
 			privateKey: privateKey,
 			domain:     "lego.acme",
 			mustStaple: true,
 			expected:   expected{len: 245},
 		},
 		{
-			desc:       "without SAN",
+			desc:       "without SAN (empty)",
 			privateKey: privateKey,
 			domain:     "lego.acme",
 			san:        []string{},

certificate/authorization.go

@@ -12,6 +12,7 @@ const (
 	// limited on the "new-reg", "new-authz" and "new-cert" endpoints.
 	// From the documentation the limitation is 20 requests per second,
 	// but using 20 as value doesn't work but 18 do.
+	// https://letsencrypt.org/docs/rate-limits/
 	overallRequestLimit = 18
 )
 

challenge/dns01/nameserver.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 	"net"
 	"os"
+	"slices"
 	"strconv"
 	"strings"
 	"sync"
@@ -216,12 +217,10 @@ func fetchSoaByFqdn(fqdn string, nameservers []string) (*soaCacheEntry, error) {
 
 // dnsMsgContainsCNAME checks for a CNAME answer in msg.
 func dnsMsgContainsCNAME(msg *dns.Msg) bool {
-	for _, ans := range msg.Answer {
+	return slices.ContainsFunc(msg.Answer, func(rr dns.RR) bool {
-		if _, ok := ans.(*dns.CNAME); ok {
+		_, ok := rr.(*dns.CNAME)
-			return true
+		return ok
-		}
+	})
-	}
-	return false
 }
 
 func dnsQuery(fqdn string, rtype uint16, nameservers []string, recursive bool) (*dns.Msg, error) {

docs/Makefile

@@ -1,6 +1,6 @@
 .PHONY: default clean hugo hugo-build
 
-default: hugo
+default: clean hugo
 
 clean:
 	rm -rf public/

platform/tester/env.go

@@ -3,6 +3,7 @@ package tester
 import (
 	"fmt"
 	"os"
+	"slices"
 )
 
 // EnvTest Environment variables manager for tests.
@@ -143,10 +144,5 @@ func (e *EnvTest) Apply(envVars map[string]string) {
 }
 
 func (e *EnvTest) isManagedKey(varName string) bool {
-	for _, key := range e.keys {
+	return slices.Contains(e.keys, varName)
-		if key == varName {
-			return true
-		}
-	}
-	return false
 }

providers/dns/cloudxns/internal/client.go

@@ -60,7 +60,7 @@ func (c *Client) GetDomainInformation(ctx context.Context, fqdn string) (*Data,
 
 	authZone, err := dns01.FindZoneByFqdn(fqdn)
 	if err != nil {
-		return nil, fmt.Errorf("cloudflare: could not find zone for FQDN %q: %w", fqdn, err)
+		return nil, fmt.Errorf("could not find zone for FQDN %q: %w", fqdn, err)
 	}
 
 	var domains []Data

providers/dns/constellix/constellix.go

@@ -6,6 +6,7 @@ import (
 	"errors"
 	"fmt"
 	"net/http"
+	"slices"
 	"strconv"
 	"time"
 
@@ -272,13 +273,11 @@ func containsValue(record *internal.Record, value string) bool {
 		return false
 	}
 
-	for _, val := range record.Value {
+	qValue := fmt.Sprintf(`%q`, value)
-		if val.Value == fmt.Sprintf(`%q`, value) {
-			return true
-		}
-	}
 
-	return false
+	return slices.ContainsFunc(record.Value, func(val internal.RecordValue) bool {
+		return val.Value == qValue
+	})
 }
 
 func backoff(min, max time.Duration, attemptNum int, resp *http.Response) time.Duration {

providers/dns/edgedns/edgedns.go

@@ -4,6 +4,7 @@ package edgedns
 import (
 	"errors"
 	"fmt"
+	"slices"
 	"strings"
 	"time"
 
@@ -224,13 +225,9 @@ func getZone(domain string) (string, error) {
 }
 
 func containsValue(values []string, value string) bool {
-	for _, val := range values {
+	return slices.ContainsFunc(values, func(val string) bool {
-		if strings.Trim(val, `"`) == value {
+		return strings.Trim(val, `"`) == value
-			return true
+	})
-		}
-	}
-
-	return false
 }
 
 func isNotFound(err error) bool {

providers/dns/oraclecloud/configprovider.go

@@ -28,7 +28,7 @@ func (p *configProvider) PrivateRSAKey() (*rsa.PrivateKey, error) {
 		return nil, err
 	}
 
-	return common.PrivateKeyFromBytes(privateKey, common.String(p.privateKeyPassphrase))
+	return common.PrivateKeyFromBytesWithPassword(privateKey, []byte(p.privateKeyPassphrase))
 }
 
 func (p *configProvider) KeyID() (string, error) {

providers/dns/yandexcloud/yandexcloud.go

@@ -7,6 +7,7 @@ import (
 	"encoding/json"
 	"errors"
 	"fmt"
+	"slices"
 	"strings"
 	"time"
 
@@ -309,10 +310,8 @@ func decodeCredentials(accountB64 string) (ycsdk.Credentials, error) {
 }
 
 func appendRecordSetData(record *ycdns.RecordSet, value string) bool {
-	for _, data := range record.GetData() {
+	if slices.Contains(record.GetData(), value) {
-		if data == value {
+		return false
-			return false
-		}
 	}
 
 	record.SetData(append(record.GetData(), value))