python-mobius 0.10.2__tar.gz → 0.10.3__tar.gz

{python_mobius-0.10.2 → python_mobius-0.10.3}/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.10.3
+
+- Frames that fail to parse or fail their CRC are logged at debug level.
+
 ## 0.10.2
 
 - Mesh peer discovery can raise its error (`raise_errors=True`) instead

{python_mobius-0.10.2 → python_mobius-0.10.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: python-mobius
-Version: 0.10.2
+Version: 0.10.3
 Summary: Reverse-engineered Python client for the Mobius BLE protocol (EcoTech Marine VorTech/Radion, AquaIllumination, Neptune Systems, NYOS)
 Project-URL: Homepage, https://code.r3pek.org/r3pek/python-mobius
 Project-URL: Documentation, https://code.r3pek.org/r3pek/python-mobius/src/branch/main/documentation

{python_mobius-0.10.2 → python_mobius-0.10.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "python-mobius"
-version = "0.10.2"
+version = "0.10.3"
 description = "Reverse-engineered Python client for the Mobius BLE protocol (EcoTech Marine VorTech/Radion, AquaIllumination, Neptune Systems, NYOS)"
 readme = "README.md"
 license = { text = "GPL-2.0-only" }

{python_mobius-0.10.2 → python_mobius-0.10.3}/src/mobius/__init__.py

@@ -87,7 +87,7 @@ from .mob import (
     MAX_PUMP_PRIMITIVE_SIZE,
 )
 
-__version__ = "0.10.2"
+__version__ = "0.10.3"
 
 __all__ = [
     "__version__",

{python_mobius-0.10.2 → python_mobius-0.10.3}/src/mobius/device.py

@@ -12,6 +12,7 @@ from __future__ import annotations
 import asyncio
 import datetime
 import struct
+import logging
 import time
 from dataclasses import dataclass, field
 from typing import Optional, Callable, Any
@@ -79,6 +80,8 @@ CHAR_TX_DATA = "01ff0103-ba5e-f4ee-5ca1-eb1e5e4b1ce0"   # write: app -> device,
 CHAR_TX_FINAL = "01ff0104-ba5e-f4ee-5ca1-eb1e5e4b1ce0"  # write: app -> device, last fragment
 
 # count=0xFFFF requests every element of a list attribute.
+_LOGGER = logging.getLogger(__name__)
+
 ALL_ELEMENTS = 0xFFFF
 
 # Write fragment size when the ATT MTU is unknown or the minimum (23).
@@ -87,6 +90,23 @@ DEFAULT_WRITE_SIZE = 20
 MTU_WRITE_OVERHEAD = 5
 
 
+def _log_bad_frame(raw: bytes, frame, trace: list[tuple[str, int]]) -> None:
+    """Debug details of a received frame that doesn't parse or fails its
+    CRC: what was received against what its header declares, and the
+    notifications it was assembled from (D = RX_DATA, F = RX_FINAL)."""
+    if not _LOGGER.isEnabledFor(logging.DEBUG):
+        return
+    declared = struct.unpack_from("<H", raw, 7)[0] + 11 if len(raw) >= 9 else None
+    pieces = " ".join(f"{kind}{length}" for kind, length in trace)
+    _LOGGER.debug(
+        "Received a frame that %s: %d bytes, header declares %s; op_group=%s op_code=%s msg_id=%s; "
+        "%d notifications [%s]; first bytes %s; last bytes %s",
+        "doesn't parse" if frame is None else "fails its CRC", len(raw), declared,
+        f"{frame.op_group:#04x}" if frame else "?", frame.op_code if frame else "?",
+        frame.message_id if frame else "?", len(trace), pieces, raw[:24].hex(), raw[-24:].hex(),
+    )
+
+
 def write_size_for_mtu(att_mtu: Optional[int]) -> int:
     """Write fragment size for a negotiated ATT MTU: `att_mtu` minus 5, as
     the app does, or DEFAULT_WRITE_SIZE when it is unknown or not above the
@@ -917,6 +937,9 @@ class MobiusDevice:
         self.adapter = adapter
         self._client: Optional[BleakClient] = None
         self._rx_buffer = bytearray()
+        # ("D"/"F", length) of each notification of the frame being
+        # received, logged when it fails its CRC.
+        self._rx_trace: list[tuple[str, int]] = []
         # Pending direct requests, keyed by FSCI message ID.
         self._pending: dict[int, asyncio.Future] = {}
         # Pending relayed requests, keyed by CoAP token (see mobius.relay).
@@ -1014,12 +1037,18 @@ class MobiusDevice:
 
     def _on_rx_data(self, _handle, data: bytearray):
         self._rx_buffer.extend(data)
+        self._rx_trace.append(("D", len(data)))
 
     def _on_rx_final(self, _handle, data: bytearray):
         self._rx_buffer.extend(data)
+        self._rx_trace.append(("F", len(data)))
         raw = bytes(self._rx_buffer)
+        trace = self._rx_trace
         self._rx_buffer.clear()
+        self._rx_trace = []
         frame = parse_frame(raw)
+        if frame is None or not frame.crc_ok:
+            _log_bad_frame(raw, frame, trace)
         if frame is None:
             return
         if self.on_any_frame is not None:

{python_mobius-0.10.2 → python_mobius-0.10.3}/tests/test_relay.py

@@ -819,3 +819,17 @@ class TestRelayedMultiBlockResponse:
         versions = await relayed.get_firmware_versions(model=Model.RadionXR15wG6Pro)
         assert versions["Product OS"] == "1.0"
         assert versions["Radio Firmware"] == "1.5.103"
+
+
+def test_a_frame_failing_its_crc_is_logged_with_its_notifications(caplog):
+    caplog.set_level(logging.DEBUG, logger="mobius.device")
+    device = MobiusDevice(BLEDevice("AA:BB:CC:DD:EE:FF", "MOBIUS", {}))
+    frame = bytearray(build_frame(OPGROUP_C2CI_REQUEST, 1, b"\x00" * 30, 7))
+    frame[-1] ^= 0xFF  # corrupt the CRC
+
+    device._on_rx_data(None, frame[:20])
+    device._on_rx_final(None, frame[20:])
+
+    assert "fails its CRC" in caplog.text
+    assert "2 notifications [D20 F" in caplog.text
+    assert device._rx_trace == []