From ff9fd553c5f913d9660c384e403502f9e60dfa86 Mon Sep 17 00:00:00 2001
From: Bernd Storath <32197462+kaaax0815@users.noreply.github.com>
Date: Wed, 2 Apr 2025 08:49:04 +0200
Subject: [PATCH] Fix: sync / rekey issue (#1789)

only sync if needed
---
 src/server/utils/WireGuard.ts | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/server/utils/WireGuard.ts b/src/server/utils/WireGuard.ts
index e827602a..6c14fc68 100644
--- a/src/server/utils/WireGuard.ts
+++ b/src/server/utils/WireGuard.ts
@@ -203,6 +203,7 @@ class WireGuard {
 
   async cronJob() {
     const clients = await Database.clients.getAll();
+    let needsSave = false;
     // Expires Feature
     for (const client of clients) {
       if (client.enabled !== true) continue;
@@ -212,6 +213,7 @@ class WireGuard {
       ) {
         WG_DEBUG(`Client ${client.id} expired.`);
         await Database.clients.toggle(client.id, false);
+        needsSave = true;
       }
     }
     // One Time Link Feature
@@ -222,10 +224,13 @@ class WireGuard {
       ) {
         WG_DEBUG(`OneTimeLink for Client ${client.id} expired.`);
         await Database.oneTimeLinks.delete(client.id);
+        // otl does not need wireguard sync
       }
     }
 
-    await this.saveConfig();
+    if (needsSave) {
+      await this.saveConfig();
+    }
   }
 }