From 9dbfc3216c936467dd91e4f777299828cbca2b62 Mon Sep 17 00:00:00 2001
From: rcourtman <courtmanr@gmail.com>
Date: Tue, 2 Dec 2025 15:24:40 +0000
Subject: [PATCH] perf: Remove redundant fmt.Sprintf in tempproxy contains
 function

The fmt.Sprintf("%v", s) call was pointless since s is already a string.
Removing this unnecessary conversion.
---
 internal/tempproxy/client.go | 1 -
 1 file changed, 1 deletion(-)

diff --git a/internal/tempproxy/client.go b/internal/tempproxy/client.go
index b5ac2209e..5a17e0271 100644
--- a/internal/tempproxy/client.go
+++ b/internal/tempproxy/client.go
@@ -213,7 +213,6 @@ func classifyError(err error, respError string) *ProxyError {
 
 // contains checks if any of the substrings are in the main string (case-insensitive)
 func contains(s string, substrs ...string) bool {
-	s = fmt.Sprintf("%v", s)
 	for _, substr := range substrs {
 		if len(s) >= len(substr) {
 			for i := 0; i <= len(s)-len(substr); i++ {