Skip to content

Added error handling for invalid or corrupt address in sensor responses & fixed _ClimateParser incorrect data format#92

Merged
raetha merged 1 commit intoraetha:develfrom
sdegeorgio:devel
Nov 11, 2025
Merged

Added error handling for invalid or corrupt address in sensor responses & fixed _ClimateParser incorrect data format#92
raetha merged 1 commit intoraetha:develfrom
sdegeorgio:devel

Conversation

@sdegeorgio
Copy link
Copy Markdown

@sdegeorgio sdegeorgio commented Nov 10, 2025

This pull request adds exception handling for three issues I experienced when I tried using bridge_tool_cli.py.

  1. When opening the /dev/hidraw0 the Dongle would send an event log with timestamp=0 (clock not yet synchronised) which would throw an exception immediately on startup.
  2. I had a sensor which had a corrupt MAC address programmed into it. When attempting to scan / pair it would cause the script to throw various exceptions and these are all addressed.
  3. Fixed _ClimateParser incorrect data format, an exception was previously being thrown when a climate event occurred.

…es & fixed _ClimateParser incorrect data format
@raetha raetha self-requested a review November 11, 2025 22:01
@raetha raetha changed the base branch from master to devel November 11, 2025 22:01
@raetha
Copy link
Copy Markdown
Owner

raetha commented Nov 11, 2025

@sdegeorgio I'm going to merge this into the devel branch now. If you could do one final check from that branch that everything still works as expected I'll roll this into the master branch shortly after.

@raetha raetha merged commit 23190cc into raetha:devel Nov 11, 2025
2 of 3 checks passed
@sdegeorgio
Copy link
Copy Markdown
Author

I've just tested the bridge_tool_cli and its working with no crashes but the leak/humidity sensor alarm is not handled, and this is the output:
Action:WARNING 2025-11-12 11:13:54,751 Unexpected sensor (03) for event E8
[2025-11-12 11:13:54][79F732CE]: type=unknown:E8

I've also tested the devel branch copied into the docker container to run with HA, with and without this pull request and I am seeing a crash when my leak/humidity sensor sends a humidity alarm, whereas on master the crash does not happen. I'll dig into it some more, but I probably wouldn't promote until this is fixed.

FYI I am using WyzeSense V1 NEOS door/window, PIR and leak/humidity sensors and dongle.

@raetha
Copy link
Copy Markdown
Owner

raetha commented Nov 12, 2025 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants