Merge pull request #154 from Snuffy2/Don't-show-error-if-OpenVPN-isn'…

…t-used- Don't show error if OpenVPN isn't used
travisghansen · Sep 6, 2024 · 3708e63 · 3708e63
2 parents c121b41 + 25e09ec
commit 3708e63
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/custom_components/opnsense/pyopnsense/__init__.py b/custom_components/opnsense/pyopnsense/__init__.py
@@ -1205,6 +1205,12 @@ def _get_telemetry_filesystems(self) -> dict:
     def _get_telemetry_openvpn(self) -> dict:
         openvpn_info: Mapping[str, Any] = self._post("/api/openvpn/export/providers")
         _LOGGER.debug(f"[get_telemetry_openvpn] openvpn_info: {openvpn_info}")
+        if (
+            openvpn_info is not None
+            and isinstance(openvpn_info, list)
+            and len(openvpn_info) == 0
+        ):
+            return {}
         if (
             openvpn_info is None
             or not isinstance(openvpn_info, Mapping)

diff --git a/custom_components/opnsense/sensor.py b/custom_components/opnsense/sensor.py
@@ -501,7 +501,7 @@ def native_value(self):
         property = self._opnsense_get_interface_property_name()
         try:
             return interface[property]
-        except KeyError:
+        except (KeyError, TypeError):
             return STATE_UNKNOWN