From 7702fa4f6bc996a2fa4a21407f120dba8670bc9c Mon Sep 17 00:00:00 2001
From: David Trattnig <david.trattnig@o94.at>
Date: Thu, 27 Feb 2020 20:11:15 +0100
Subject: [PATCH] Improved logs for defect detection.

---
 modules/communication/liquidsoap/client.py     | 11 +++++++----
 modules/communication/liquidsoap/initthread.py |  5 +++--
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/modules/communication/liquidsoap/client.py b/modules/communication/liquidsoap/client.py
index 6aa8e593..58be72fc 100644
--- a/modules/communication/liquidsoap/client.py
+++ b/modules/communication/liquidsoap/client.py
@@ -30,7 +30,7 @@ import logging
 from multiprocessing import Lock
 
 from libraries.exceptions.auraexceptions import LQConnectionError
-
+from libraries.enum.auraenumerations import TerminalColors
 
 """ 
     LiquidSoapClient Class 
@@ -75,7 +75,7 @@ class LiquidSoapClient:
             self.socket.connect(self.socket_path)
         except socket.error as e:
             msg = "Cannot connect to socketpath " + self.socket_path + ". Reason: "+str(e)
-            self.logger.critical(msg)
+            self.logger.critical(TerminalColors.RED.value+msg+TerminalColors.ENDC.value)
             self.can_connect = False
             self.connected = False
 #            raise e
@@ -84,6 +84,9 @@ class LiquidSoapClient:
             self.connected = True
             return True
 
+
+# AttributeError('characters_written')
+
     # ------------------------------------------------------------------------------------------ #
     def is_connected(self):
         return self.connected
@@ -131,7 +134,7 @@ class LiquidSoapClient:
             self.mutex.release()
 
         except Exception as e:
-            self.logger.error(str(e))
+            self.logger.error(TerminalColors.RED.value+str(e)+TerminalColors.ENDC.value)
             self.mutex.release()
 
         return data
@@ -211,7 +214,7 @@ class LiquidSoapClient:
                 if not self.disable_logging:
                     self.logger.info("LiquidSoapClient got reply: " + self.message)
             except BrokenPipeError as e:
-                self.logger.error("Detected a problem with liquidsoap connection while sending: " + message + ". Reason: " + str(e) + "! Trying to reconnect.")
+                self.logger.error(TerminalColors.RED.value+"Detected a problem with liquidsoap connection while sending: " + message + ". Reason: " + str(e) + "! Trying to reconnect."+TerminalColors.RED.value)
                 self.connect()
                 raise
 
diff --git a/modules/communication/liquidsoap/initthread.py b/modules/communication/liquidsoap/initthread.py
index 4e92586c..9ec35f6b 100644
--- a/modules/communication/liquidsoap/initthread.py
+++ b/modules/communication/liquidsoap/initthread.py
@@ -27,7 +27,8 @@ import logging
 import datetime
 import threading
 
-from libraries.enum.auraenumerations import ScheduleEntryType
+from libraries.enum.auraenumerations import ScheduleEntryType, TerminalColors
+
 
 """
 LiquidSoapInitThread class.
@@ -74,7 +75,7 @@ class LiquidSoapInitThread(threading.Thread):
             self.liquidsoapcommunicator.is_liquidsoap_running = True
 
         except Exception as e:
-            self.logger.critical("Liquidsoap connection ERROR! Restart LQ Server! Reason: "+str(e), e)
+            self.logger.critical(TerminalColors.RED.value+"Liquidsoap connection ERROR! Restart LQ Server! Reason: "+str(e)+TerminalColors.ENDC.value)
 
 
 
-- 
GitLab