search

bropat / eufy-security-client

This shared library allows to control Eufy security devices by connecting to the Eufy cloud servers and local/remote stations over p2p.
MIT License
480 stars 69 forks source link

[Bug]: Motion detection is triggered on Vehicle and Face detection #489

Closed martijnpoppen closed 1 day ago

martijnpoppen commented 5 months ago

Client version

latest

Node version

20

Operating System type

Other

Operating system version

Homey

Describe the bug

Hi @bropat Since the last update I get reports from people that the Motion detection is also triggered when there's Face or Vehicle detection.

I saw this commit: https://github.com/bropat/eufy-security-client/commit/47f24e7f089ac1af15ed4d2fc6ed7de6078cbae7

What's the reasoning behind this? Because this is contradicting with the official Eufy app. Because there it's only a Face or a Vehicle detection. Motion detection is a separate thing.

For now I reverted this in my integration as it's breaking a lot of flows/automations for people

To reproduce

Get a vehicle or face detection

Screenshots & Logfiles

No response

Additional context

No response

martijnpoppen commented 5 months ago

@bropat not sure if you like it, but this might be a potential fix: #494