Update firewall.py: do not actually enable upnp if no upnp device is …

…available on the network
YunoHost · Mar 2, 2025 · 207de42 · 207de42
1 parent 5b127c7
commit 207de42
Showing 1 changed file with 3 additions and 0 deletions.
diff --git 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(