Handle all construct parse errors the same

nibe/connection/nibegw.py

@@ -19,8 +19,8 @@
     BitStruct,
     Bytes,
     Checksum,
-    ChecksumError,
     Const,
+    ConstructError,
     Container,
     Enum,
     EnumIntegerString,
@@ -210,16 +210,15 @@
                 )
             elif not isinstance(cmd, EnumIntegerString):
                 logger.debug(f"Unknown command {cmd}")
-        except ChecksumError:
+        except ConstructError as e:
             logger.warning(
-                f"Ignoring packet from {addr} due to checksum error: {hexlify(data).decode('utf-8')}"
+                f"Ignoring packet from {addr} due to parse error: {hexlify(data).decode('utf-8')}: {e}"
             )
         except NibeException as e:
             logger.error(f"Failed handling packet from {addr}: {e}")
-        except Exception as e:
+        except Exception:
             logger.exception(
-                f"Unexpected exception during parsing packet data '{hexlify(data).decode('utf-8')}' from {addr}",
-                e,
+                f"Unexpected exception during parsing packet data '{hexlify(data).decode('utf-8')}' from {addr}"
             )
 
     async def read_product_info(

nibe/console_scripts/cli.py

@@ -9,7 +9,16 @@
 from typing import IO
 
 import asyncclick as click
-from construct import Const, GreedyRange, Int8ul, RawCopy, Select, Struct, Terminated
+from construct import (
+    Const,
+    ConstructError,
+    GreedyRange,
+    Int8ul,
+    RawCopy,
+    Select,
+    Struct,
+    Terminated,
+)
 
 from ..coil import CoilData
 from ..connection import Connection
@@ -191,8 +200,12 @@
         raw = bytes.fromhex(data)
     elif type == "bytes":
         raw = bytes(literal_eval(data))
-    request = Block.parse(raw)
-    click.echo(request)
+
+    try:
+        request = Block.parse(raw)
+        click.echo(request)
+    except ConstructError as exception:
+        click.echo(f"Failed to parse: {exception}", err=True)
 
 
 def read_bytes_socat(file: IO):