Logging issue workaround for fritzconnection library (#60448)

* Logging issue workaround * Better approach
home-assistant · Nov 27, 2021 · 3cd80b9 · 3cd80b9
1 parent bb99d07
commit 3cd80b9
Show file tree

Hide file tree

Showing 6 changed files with 20 additions and 4 deletions.
diff --git a/homeassistant/components/fritz/__init__.py b/homeassistant/components/fritz/__init__.py
@@ -2,6 +2,7 @@
 import logging
 
 from fritzconnection.core.exceptions import FritzConnectionException, FritzSecurityError
+from fritzconnection.core.logger import fritzlogger
 
 from homeassistant.config_entries import ConfigEntry
 from homeassistant.const import (
@@ -20,6 +21,13 @@
 
 _LOGGER = logging.getLogger(__name__)
 
+level = _LOGGER.getEffectiveLevel()
+_LOGGER.info(
+    "Setting logging level of fritzconnection: %s", logging.getLevelName(level)
+)
+fritzlogger.set_level(level)
+fritzlogger.enable()
+
 
 async def async_setup_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
     """Set up fritzboxtools from config entry."""

diff --git a/homeassistant/components/fritz/manifest.json b/homeassistant/components/fritz/manifest.json
@@ -3,7 +3,7 @@
   "name": "AVM FRITZ!Box Tools",
   "documentation": "https://www.home-assistant.io/integrations/fritz",
   "requirements": [
-    "fritzconnection==1.7.0",
+    "fritzconnection==1.7.2",
     "xmltodict==0.12.0"
   ],
   "dependencies": ["network"],

diff --git a/homeassistant/components/fritzbox_callmonitor/__init__.py b/homeassistant/components/fritzbox_callmonitor/__init__.py
@@ -2,6 +2,7 @@
 import logging
 
 from fritzconnection.core.exceptions import FritzConnectionException, FritzSecurityError
+from fritzconnection.core.logger import fritzlogger
 from requests.exceptions import ConnectionError as RequestsConnectionError
 
 from homeassistant.const import CONF_HOST, CONF_PASSWORD, CONF_USERNAME
@@ -19,6 +20,13 @@
 
 _LOGGER = logging.getLogger(__name__)
 
+level = _LOGGER.getEffectiveLevel()
+_LOGGER.info(
+    "Setting logging level of fritzconnection: %s", logging.getLevelName(level)
+)
+fritzlogger.set_level(level)
+fritzlogger.enable()
+
 
 async def async_setup_entry(hass, config_entry):
     """Set up the fritzbox_callmonitor platforms."""

diff --git a/homeassistant/components/fritzbox_callmonitor/manifest.json b/homeassistant/components/fritzbox_callmonitor/manifest.json
@@ -3,7 +3,7 @@
   "name": "AVM FRITZ!Box Call Monitor",
   "config_flow": true,
   "documentation": "https://www.home-assistant.io/integrations/fritzbox_callmonitor",
-  "requirements": ["fritzconnection==1.7.0"],
+  "requirements": ["fritzconnection==1.7.2"],
   "codeowners": [],
   "iot_class": "local_polling"
 }
diff --git a/requirements_all.txt b/requirements_all.txt
@@ -680,7 +680,7 @@ freesms==0.2.0
 
 # homeassistant.components.fritz
 # homeassistant.components.fritzbox_callmonitor
-fritzconnection==1.7.0
+fritzconnection==1.7.2
 
 # homeassistant.components.google_translate
 gTTS==2.2.3

diff --git a/requirements_test_all.txt b/requirements_test_all.txt
@@ -415,7 +415,7 @@ freebox-api==0.0.10
 
 # homeassistant.components.fritz
 # homeassistant.components.fritzbox_callmonitor
-fritzconnection==1.7.0
+fritzconnection==1.7.2
 
 # homeassistant.components.google_translate
 gTTS==2.2.3