fix(lint): avoid string concat in loop

Signed-off-by: Chris Gianelloni <[email protected]>

Author: wolf31o2

internal/dns/dns.go

@@ -552,47 +552,49 @@ func randomFallbackServer() string {
 }
 
 func formatMessageAnswerSection(section []dns.RR) string {
-	ret := "[ "
+	var sb strings.Builder
+	sb.WriteString("[ ")
 	for idx, rr := range section {
-		ret += fmt.Sprintf(
-			"< %s >",
-			strings.ReplaceAll(
-				strings.TrimPrefix(
-					rr.String(),
-					";",
-				),
-				"\t",
-				" ",
+		sb.WriteString("< ")
+		sb.WriteString(strings.ReplaceAll(
+			strings.TrimPrefix(
+				rr.String(),
+				";",
 			),
+			"\t",
+			" ",
+		),
 		)
+		sb.WriteString(" >")
 		if idx != len(section)-1 {
-			ret += `,`
+			sb.WriteString(`,`)
 		}
-		ret += ` `
+		sb.WriteString(` `)
 	}
-	ret += "]"
-	return ret
+	sb.WriteString("]")
+	return sb.String()
 }
 
 func formatMessageQuestionSection(section []dns.Question) string {
-	ret := "[ "
+	var sb strings.Builder
+	sb.WriteString("[ ")
 	for idx, question := range section {
-		ret += fmt.Sprintf(
-			"< %s >",
-			strings.ReplaceAll(
-				strings.TrimPrefix(
-					question.String(),
-					";",
-				),
-				"\t",
-				" ",
+		sb.WriteString("< ")
+		sb.WriteString(strings.ReplaceAll(
+			strings.TrimPrefix(
+				question.String(),
+				";",
 			),
+			"\t",
+			" ",
+		),
 		)
+		sb.WriteString(" >")
 		if idx != len(section)-1 {
-			ret += `,`
+			sb.WriteString(`,`)
 		}
-		ret += ` `
+		sb.WriteString(` `)
 	}
-	ret += "]"
-	return ret
+	sb.WriteString("]")
+	return sb.String()
 }

internal/handshake/domain.go

@@ -13,6 +13,7 @@ import (
 	"fmt"
 	"io"
 	"net"
+	"strings"
 )
 
 const (
@@ -282,8 +283,8 @@ func domainRecordIPv6Decode(r *BytesReader) (net.IP, error) {
 }
 
 func domainRecordNameDecode(r *BytesReader) (string, error) {
+	var sb strings.Builder
 	// NOTE: this function is mostly ported straight from hnsd
-	var name string
 	for {
 		c, err := r.ReadByte()
 		if err != nil {
@@ -311,10 +312,10 @@ func domainRecordNameDecode(r *BytesReader) (string, error) {
 				if b == 0x2e {
 					b = 0xfe
 				}
-				name += string([]byte{b})
+				sb.WriteString(string([]byte{b}))
 			}
-			if len(name) > 0 {
-				name += "."
+			if len(sb.String()) > 0 {
+				sb.WriteString(".")
 			}
 		case 0xc0:
 			// Lookup name from earlier in the buffer
@@ -338,5 +339,5 @@ func domainRecordNameDecode(r *BytesReader) (string, error) {
 			return "", errors.New("unexpected value")
 		}
 	}
-	return name, nil
+	return sb.String(), nil
 }