From 6349fc6502646d3293e8d1160678a55bd04f47e7 Mon Sep 17 00:00:00 2001
From: Jacob Kiers <code@kiers.eu>
Date: Fri, 25 Aug 2023 22:52:46 +0200
Subject: [PATCH] Prevent unnecessary clone

This also ensures that the address resolver actually keeps state.
Otherwise it was cloned before each resolution, resulting in it never
keeping the resolved addresses.

Signed-off-by: Jacob Kiers <code@kiers.eu>
---
 src/servers/protocol/tcp.rs | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/src/servers/protocol/tcp.rs b/src/servers/protocol/tcp.rs
index f33a7f8..f4565f6 100644
--- a/src/servers/protocol/tcp.rs
+++ b/src/servers/protocol/tcp.rs
@@ -72,21 +72,17 @@ async fn accept(inbound: TcpStream, proxy: Arc<Proxy>) -> Result<(), Box<dyn std
                 "No upstream named {:?} on server {:?}",
                 proxy.default_action, proxy.name
             );
-            return process(
-                inbound,
-                proxy.upstream.get(&proxy.default_action).unwrap().clone(),
-            )
-            .await;
+            return process(inbound, proxy.upstream.get(&proxy.default_action).unwrap()).await;
             // ToDo: Remove unwrap and check default option
         }
     };
 
-    return process(inbound, upstream.clone()).await;
+    return process(inbound, &upstream).await;
 }
 
 async fn process(
     mut inbound: TcpStream,
-    upstream: Upstream,
+    upstream: &Upstream,
 ) -> Result<(), Box<dyn std::error::Error>> {
     match upstream {
         Upstream::Ban => {