Added logging

ParadoxAlarmInterface · Aug 26, 2024 · 18f3de9 · 18f3de9
1 parent 1cd2964
commit 18f3de9
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/paradox/interfaces/text/homeassistant_notifications.py b/paradox/interfaces/text/homeassistant_notifications.py
@@ -74,7 +74,7 @@ def send_message(self, message: str, level: EventLevel):
 
         res = requests.post(url, json=payload, headers=headers)
         if res.status_code == 200:
-            logger.debug(f"Notification sent: {message}, level={level}")
+            logger.info(f"Notification sent: {message}, level={level}")
         else:
             logger.error(
                 f"Failed to send notification: code={res.status_code}, text: {res.text}"

diff --git a/paradox/interfaces/text/pushover.py b/paradox/interfaces/text/pushover.py
@@ -1,5 +1,3 @@
-# -*- coding: utf-8 -*-
-
 import logging
 import re
 
@@ -37,7 +35,7 @@ def __init__(self, alarm):
         self.users = {}
 
     def _run(self):
-        super(PushoverTextInterface, self)._run()
+        super()._run()
 
         self.app = chump.Application(cfg.PUSHOVER_KEY)
         if not self.app.is_authenticated:
@@ -50,7 +48,7 @@ def send_message(self, message: str, level: EventLevel):
             user_key = settings["user_key"]
             devices_raw = settings["devices"]
 
-            user = self.users.get(user_key)  # type: chump.User
+            user = self.users.get(user_key)
 
             if user is None:
                 user = self.users[user_key] = self.app.get_user(user_key)
@@ -68,7 +66,8 @@ def send_message(self, message: str, level: EventLevel):
                         title="Alarm",
                         priority=_level_2_priority.get(level, chump.NORMAL),
                     )
-                except:
+                    logger.info(f"Notification sent: {message}, level={level}")
+                except Exception:
                     logger.exception("Pushover send message")
 
             else:
@@ -88,7 +87,10 @@ def send_message(self, message: str, level: EventLevel):
                             device=device,
                             priority=_level_2_priority.get(level, chump.NORMAL),
                         )
-                    except:
+                        logger.info(
+                            f"Notification sent: {message}, level={level}, device={device}"
+                        )
+                    except Exception:
                         logger.exception("Pushover send message")
 
         # TODO: Missing the message reception