From 4a504468d3883c9d694cdc56dbb6971edd466877 Mon Sep 17 00:00:00 2001
From: Simon <simon@gb7fr.org.uk>
Date: Tue, 31 Jan 2023 10:19:48 +0000
Subject: [PATCH] Debug when systems have no bridges.

Only enable when debugging.
---
 bridge_master.py | 18 ++++++++++++++++++
 config.py        |  3 ++-
 2 files changed, 20 insertions(+), 1 deletion(-)

diff --git a/bridge_master.py b/bridge_master.py
index bd73887..8fb0a63 100644
--- a/bridge_master.py
+++ b/bridge_master.py
@@ -388,6 +388,17 @@ def statTrimmer():
     if CONFIG['REPORTS']['REPORT']:
         report_server.send_clients(b'bridge updated')
 
+#Identify systems with no bridges
+def bridgeDebug():
+    for system in CONFIG['SYSTEMS']:
+        bridgeroll = 0
+        for bridge in BRIDGES:
+            for enabled_system in BRIDGE['bridge']:
+                if enabled_system == system:
+                    bridgeroll += 1
+        if not bridgeroll:
+            logger.warn('{BRIDGEDEBUG) system %s has no bridges', system)
+
 def kaReporting():
     logger.debug('(ROUTER) KeepAlive reporting loop started')
     for system in systems:
@@ -2757,6 +2768,13 @@ if __name__ == '__main__':
     ka_task = task.LoopingCall(kaReporting)
     ka = ka_task.start(60)
     ka.addErrback(loopingErrHandle)
+
+    #Debug bridges
+    if CONFIG['GLOBAL']['DEBUG_BRIDGES']:
+        debug_bridges_task = task.LoopingCall(bridgeDebug)
+        debug_bridges = debug_bridges_task.start(66)
+        debug_bridges.addErrback(loopingErrHandle)
+
     
     #Subscriber map trimmer
     sub_trimmer_task = task.LoopingCall(SubMapTrimmer)
diff --git a/config.py b/config.py
index 3eed4e2..2e51569 100755
--- a/config.py
+++ b/config.py
@@ -147,7 +147,8 @@ def build_config(_config_file):
                     'ANNOUNCEMENT_LANGUAGES': config.get(section, 'ANNOUNCEMENT_LANGUAGES', fallback=''),
                     'SERVER_ID': config.getint(section, 'SERVER_ID', fallback=0).to_bytes(4, 'big'),
                     'DATA_GATEWAY': config.getboolean(section, 'DATA_GATEWAY', fallback=False),
-                    'VALIDATE_SERVER_IDS': config.getboolean(section, 'VALIDATE_SERVER_IDS', fallback=True)
+                    'VALIDATE_SERVER_IDS': config.getboolean(section, 'VALIDATE_SERVER_IDS', fallback=True),
+                    'DEBUG_BRIDGES' : config.getboolean(section, 'DEBUG_BRIDGES', fallback=False)
                     
                 })
                 if not CONFIG['GLOBAL']['ANNOUNCEMENT_LANGUAGES']: