python-izone 1.3.6__py3-none-any.whl → 1.3.8__py3-none-any.whl

pizone/__init__.py

@@ -31,7 +31,12 @@ from typing import Any
 from . import power as power_mod
 from .controller import Controller
 from .discovery import DiscoveryService, Listener, create_discovery, discovery
-from .exceptions import ControllerCommandError, ResponseDecodeError, UnpairedBridgeError
+from .exceptions import (
+    ControllerAlreadyClaimedError,
+    ControllerCommandError,
+    ResponseDecodeError,
+    UnpairedBridgeError,
+)
 from .power import BatteryLevel, Power, PowerChannel, PowerDevice, PowerGroup
 from .types import ControllerEndpoint
 from .zone import Zone
@@ -61,6 +66,7 @@ __all__ = [
     "ENABLE_POWER",
     "BatteryLevel",
     "Controller",
+    "ControllerAlreadyClaimedError",
     "ControllerCommandError",
     "ControllerEndpoint",
     "DiscoveryService",

pizone/discovery.py

@@ -24,7 +24,7 @@ import ifaddr
 
 from .const import PLACEHOLDER_DEVICE_UID
 from .controller import Controller
-from .exceptions import raise_if_placeholder_uid
+from .exceptions import ControllerAlreadyClaimedError, raise_if_placeholder_uid
 from .types import ControllerEndpoint
 from .zone import Zone
 
@@ -405,7 +405,9 @@ class DiscoveryService:
     def _ensure_endpoint_available(self, endpoint: ControllerEndpoint) -> None:
         """Raise if *endpoint* is already claimed."""
         if endpoint.uid in self._claimed_endpoints:
-            raise RuntimeError(f"Controller {endpoint.uid} already created")
+            raise ControllerAlreadyClaimedError(
+                f"Controller {endpoint.uid} already created"
+            )
 
     # disposition: 1.4
     def _claim_endpoint(self, endpoint: ControllerEndpoint) -> None:
@@ -451,7 +453,9 @@ class DiscoveryService:
         """
         claimed = self._endpoint_by_host(self._claimed_endpoints, host)
         if claimed is not None:
-            raise RuntimeError(f"Controller {claimed.uid} already created")
+            raise ControllerAlreadyClaimedError(
+                f"Controller {claimed.uid} already created"
+            )
         known = self._endpoint_by_host(self._known_endpoints, host)
         if known is not None:
             return known
@@ -476,7 +480,7 @@ class DiscoveryService:
         """
         raise_if_placeholder_uid(uid)
         if uid in self._claimed_endpoints:
-            raise RuntimeError(f"Controller {uid} already created")
+            raise ControllerAlreadyClaimedError(f"Controller {uid} already created")
         endpoint = self._known_endpoints.get(uid)
         if endpoint is not None:
             return endpoint
@@ -545,7 +549,7 @@ class DiscoveryService:
                     continue
                 try:
                     self._cache_endpoint(endpoint)
-                except RuntimeError:
+                except ControllerAlreadyClaimedError:
                     continue
                 verified.append(endpoint)
                 if endpoint.uid not in notified:
@@ -569,7 +573,7 @@ class DiscoveryService:
 
         Raises:
             UnpairedBridgeError: If *uid* is the unpaired placeholder (no I/O).
-            RuntimeError: If a controller with *uid* already exists.
+            ControllerAlreadyClaimedError: If a controller with *uid* already exists.
             ConnectionError: If the controller cannot be reached after
                 address fallback.
             ControllerCommandError: If the device rejects the protocol request.
@@ -579,7 +583,7 @@ class DiscoveryService:
         """
         raise_if_placeholder_uid(uid)
         if uid in self._claimed_endpoints:
-            raise RuntimeError(f"Controller {uid} already created")
+            raise ControllerAlreadyClaimedError(f"Controller {uid} already created")
         pending_address_changed: ControllerEndpoint | None = None
 
         try:
@@ -795,8 +799,13 @@ class DiscoveryService:
         """Return the aiohttp session used for HTTP requests."""
         return self._session
 
-    def _on_error_received(self, _: Exception) -> None:
-        _LOG.warning("Error passed and ignored to error_received", exc_info=True)
+    def _on_error_received(self, exc: Exception) -> None:
+        # Expected when broadcast is blocked (firewall / no permission) or the
+        # interface is down; scan just finds nothing. Keep unexpected errors loud.
+        if isinstance(exc, OSError):
+            _LOG.debug("UDP error ignored: %s", exc)
+            return
+        _LOG.warning("Error passed and ignored to error_received: %s", exc)
 
     def _find_by_addr(self, addr: tuple[str, int]) -> Controller | None:
         for ctrl in self._controllers.values():
@@ -919,8 +928,7 @@ class DiscoveryService:
         previous = self._known_endpoints.get(endpoint.uid)
         newly_seen = previous is None
         host_changed = previous is not None and previous.host != endpoint.host
-        with suppress(RuntimeError):
-            self._cache_endpoint(endpoint)
+        self._cache_endpoint(endpoint)
         if newly_seen or host_changed:
             self._emit_endpoint_discovered(endpoint)
 

pizone/exceptions.py

@@ -28,6 +28,14 @@ class UnpairedBridgeError(ValueError):
     """
 
 
+class ControllerAlreadyClaimedError(Exception):
+    """A controller UID or host is already claimed on this discovery service.
+
+    Raised when ``discover_by_host``, ``discover_by_uid``, or ``create_controller``
+    would duplicate an active controller registration.
+    """
+
+
 def raise_if_placeholder_uid(uid: str) -> None:
     """Raise :exc:`UnpairedBridgeError` if *uid* is the unpaired placeholder."""
     if uid == PLACEHOLDER_DEVICE_UID:

pizone/version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '1.3.6'
-__version_tuple__ = version_tuple = (1, 3, 6)
+__version__ = version = '1.3.8'
+__version_tuple__ = version_tuple = (1, 3, 8)
 
 __commit_id__ = commit_id = None

{python_izone-1.3.6.dist-info → python_izone-1.3.8.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-izone
-Version: 1.3.6
+Version: 1.3.8
 Summary: A python interface to the iZone airconditioner controller
 Project-URL: Homepage, https://github.com/Swamp-Ig/pizone
 Author-email: Penny Wood <pypl@ninjateaparty.com>

python_izone-1.3.8.dist-info/RECORD

@@ -0,0 +1,14 @@
+pizone/__init__.py,sha256=9EbQzqD1XfS5fSE-zgFieuzxAfZcQb9a3wNXsMp0JKc,2644
+pizone/const.py,sha256=0skTh6zBe61AJ_57fqdBrxz3aVBM3T4Ak_cPSkOx2Bo,153
+pizone/controller.py,sha256=nctzAF8Y75T0_Ue9Hxxv9OfEex4zoNseOMjGfI6wHFE,41109
+pizone/discovery.py,sha256=ZEXlzMgEcK2qYCU7UXCdgOrgv6vChOCgN_ddq5rACg0,37685
+pizone/exceptions.py,sha256=qzYNHzQBMDcuvLfxlvOsk6J2Lp0gtzlHqJWpzyEunvU,1603
+pizone/power.py,sha256=bXoDRVD6gCfq0kJv132sjfy13DHGd6p2uBg9HcxRSxA,10343
+pizone/py.typed,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
+pizone/types.py,sha256=fZrnec0Q815HyG2sFciLqNX0QDbTq1_0wTF92GXQh78,200
+pizone/version.py,sha256=FXUTXirc7-JCTXzXG9H84JqQTySUJIh6ktR1ZkEoISc,520
+pizone/zone.py,sha256=tqiLJAJCmp1cehCtVPjawUNY9qyYypC4x3igI5jtDNo,7446
+python_izone-1.3.8.dist-info/METADATA,sha256=S5q-Hty056t4AYv8gUgXUhvRbfgpuUButQxLBXAxpi0,2703
+python_izone-1.3.8.dist-info/WHEEL,sha256=lCkmxWfQsSc9CfIClYeavTdQeEX2toPqufh9gI35EQA,87
+python_izone-1.3.8.dist-info/licenses/licence.txt,sha256=OXLcl0T2SZ8Pmy2_dmlvKuetivmyPd5m1q-Gyd-zaYY,35149
+python_izone-1.3.8.dist-info/RECORD,,

python_izone-1.3.6.dist-info/RECORD

@@ -1,14 +0,0 @@
-pizone/__init__.py,sha256=yJMEJCKBU0mYOuvvBZFG5NTkpMpMWBAg9MvmzP3npK8,2555
-pizone/const.py,sha256=0skTh6zBe61AJ_57fqdBrxz3aVBM3T4Ak_cPSkOx2Bo,153
-pizone/controller.py,sha256=nctzAF8Y75T0_Ue9Hxxv9OfEex4zoNseOMjGfI6wHFE,41109
-pizone/discovery.py,sha256=C9AEK6nMyo2xU-jZ3BWyADnTMM-X1EkgNh9LaECkpBc,37266
-pizone/exceptions.py,sha256=wRO-COpyNyKL6P1vG0MlAVwbwo2iOvH0kRO7O75PUlo,1327
-pizone/power.py,sha256=bXoDRVD6gCfq0kJv132sjfy13DHGd6p2uBg9HcxRSxA,10343
-pizone/py.typed,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-pizone/types.py,sha256=fZrnec0Q815HyG2sFciLqNX0QDbTq1_0wTF92GXQh78,200
-pizone/version.py,sha256=_FUKw5XE-1Iz4eIZxskUWzdnDjBxzXHPurN7TvxgxV8,520
-pizone/zone.py,sha256=tqiLJAJCmp1cehCtVPjawUNY9qyYypC4x3igI5jtDNo,7446
-python_izone-1.3.6.dist-info/METADATA,sha256=nv6-HMl-Py3IKlUPtstcKfpEz1LbsmOZGF1P0oOy8ao,2703
-python_izone-1.3.6.dist-info/WHEEL,sha256=lCkmxWfQsSc9CfIClYeavTdQeEX2toPqufh9gI35EQA,87
-python_izone-1.3.6.dist-info/licenses/licence.txt,sha256=OXLcl0T2SZ8Pmy2_dmlvKuetivmyPd5m1q-Gyd-zaYY,35149
-python_izone-1.3.6.dist-info/RECORD,,