From 233e281fcd31451ee8ba3f463c30f45ac6e648f7 Mon Sep 17 00:00:00 2001
From: Ludo Stellingwerff <ludo.stellingwerff@gmail.com>
Date: Tue, 1 Oct 2024 10:03:34 +0200
Subject: [PATCH] Try not to cleanup multiple peers behind same PublicIP

---
 pkg/backend/wireguard/device.go | 46 ---------------------------------
 1 file changed, 46 deletions(-)

diff --git a/pkg/backend/wireguard/device.go b/pkg/backend/wireguard/device.go
index ec0364af44..ea5be953a0 100644
--- a/pkg/backend/wireguard/device.go
+++ b/pkg/backend/wireguard/device.go
@@ -306,52 +306,6 @@ func (dev *wgDevice) addPeer(publicEndpoint string, peerPublicKeyRaw string, pee
 		return fmt.Errorf("failed to configure device %w", err)
 	}
 
-	// Remove peers from this endpoint with different public keys
-	err = dev.cleanupEndpointPeers(udpEndpoint, peerPublicKeyRaw)
-	if err != nil {
-		return fmt.Errorf("failed to clean up endpoint peers %w", err)
-	}
-
-	return nil
-}
-
-func (dev *wgDevice) cleanupEndpointPeers(udpEndpoint *net.UDPAddr, latestPublicKeyRaw string) error {
-	client, err := wgctrl.New()
-	if err != nil {
-		return fmt.Errorf("failed to open wgctrl: %w", err)
-	}
-	defer client.Close()
-
-	currentDev, err := client.Device(dev.attrs.name)
-	if err != nil {
-		return fmt.Errorf("failed to open device: %w", err)
-	}
-
-	peers := []wgtypes.PeerConfig{}
-	for _, peer := range currentDev.Peers {
-		if peer.Endpoint.IP.Equal(udpEndpoint.IP) {
-			if peer.PublicKey.String() != latestPublicKeyRaw {
-				removePeer := wgtypes.PeerConfig{
-					PublicKey: peer.PublicKey,
-					Remove:    true,
-				}
-				peers = append(peers, removePeer)
-			}
-		}
-	}
-
-	wgcfg := wgtypes.Config{
-		PrivateKey:   dev.attrs.privateKey,
-		ListenPort:   &dev.attrs.listenPort,
-		ReplacePeers: false,
-		Peers:        peers,
-	}
-
-	err = client.ConfigureDevice(dev.attrs.name, wgcfg)
-	if err != nil {
-		return fmt.Errorf("failed to cleanup peers %w", err)
-	}
-
 	return nil
 }