From 207de4228cedf95854deba405e99f41c2aef9046 Mon Sep 17 00:00:00 2001
From: Alexandre Aubin <4533074+alexAubin@users.noreply.github.com>
Date: Sun, 2 Mar 2025 16:03:39 +0100
Subject: [PATCH] Update firewall.py: do not actually enable upnp if no upnp
 device is available on the network

---
 src/firewall.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/firewall.py b/src/firewall.py
index 7f1aefee32..32cd26199f 100644
--- a/src/firewall.py
+++ b/src/firewall.py
@@ -280,6 +280,9 @@ def refresh(self, firewall: "YunoFirewall") -> bool:
         return status
 
     def enable(self) -> None:
+        if not self.find_gid():
+            logger.error("Not enabling UPnP because no UPnP device was found")
+            return
         if not self.enabled():
             # Add cron job
             self.UPNP_CRON_JOB.write_text(