refactor: avoid calling strings.ToLower twice (#3433)

gin-gonic · Dec 22, 2022 · 297b664 · 297b664
1 parent 2285aa5
commit 297b664
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/recovery.go b/recovery.go
@@ -62,7 +62,9 @@ func CustomRecoveryWithWriter(out io.Writer, handle RecoveryFunc) HandlerFunc {
 				if ne, ok := err.(*net.OpError); ok {
 					var se *os.SyscallError
 					if errors.As(ne, &se) {
-						if strings.Contains(strings.ToLower(se.Error()), "broken pipe") || strings.Contains(strings.ToLower(se.Error()), "connection reset by peer") {
+						seStr := strings.ToLower(se.Error())
+						if strings.Contains(seStr, "broken pipe") ||
+							strings.Contains(seStr, "connection reset by peer") {
 							brokenPipe = true
 						}
 					}