samemory / homebridge-eufy-security

Work in progress
Apache License 2.0
97 stars 14 forks source link

[Bug]: Duplicate notification on doorbell ring #236

Closed csrui closed 2 years ago

csrui commented 3 years ago

What happened?

When pressing the doorbell once, I get two HomeKit notifications.

I have tried reconfiguring several times but the end result is always the same. Two notifications.

When setting up, I can see two doorbell items, one being a child of the other. Might be related to this.

Screenshot 2021-09-09 at 09 59 18

Device Type

Doorbell (Supported)

Plugin Version

v1.0.x (Supported)

HomeBridge Version

v1.3.x (Supported)

NodeJS Version

v14 (Supported)

Operating System

Docker

Relevant log output

[9/9/2021, 7:25:23 PM] [Homebridge UI] Homebridge log truncated by admin.
[9/9/2021, 7:26:32 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Doorbell Property Changes: ringing { value: true, timestamp: 1631211989334 }
[9/9/2021, 7:26:32 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Doorbell Property Changes: pictureUrl {
  value: 'https://security-app-eu.eufylife.com/v1/XXXXXXXX',
  timestamp: 1631211989334
}
[9/9/2021, 7:26:32 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Doorbell DoorBell ringing
[9/9/2021, 7:26:32 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot requested: 1280 x 720 Doorbell false
[9/9/2021, 7:26:32 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i https://security-app-eu.eufylife.com/v1/XXXXXXXX -frames:v 1 -f image2 - -hide_banner -loglevel error Doorbell false
[9/9/2021, 7:26:32 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot requested: 1280 x 720 Doorbell false
[9/9/2021, 7:26:32 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot requested: 1280 x 720 Doorbell false
[9/9/2021, 7:26:32 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot requested: 1280 x 720 Doorbell false
[9/9/2021, 7:26:33 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Fetching snapshot took 0.68 seconds. Doorbell false
[9/9/2021, 7:26:33 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Sending snapshot: 1280 x 720 Doorbell false
[9/9/2021, 7:26:33 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Resize command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i pipe: -frames:v 1 -filter:v scale='min(1280,iw)':'min(720,ih)':force_original_aspect_ratio=decrease -f image2 - Doorbell false
[9/9/2021, 7:26:33 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Sending snapshot: 1280 x 720 (cached) Doorbell false
[9/9/2021, 7:26:33 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Resize command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i pipe: -frames:v 1 -filter:v scale='min(1280,iw)':'min(720,ih)':force_original_aspect_ratio=decrease -f image2 - Doorbell false
[9/9/2021, 7:26:33 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Sending snapshot: 1280 x 720 (cached) Doorbell false
[9/9/2021, 7:26:33 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Resize command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i pipe: -frames:v 1 -filter:v scale='min(1280,iw)':'min(720,ih)':force_original_aspect_ratio=decrease -f image2 - Doorbell false
[9/9/2021, 7:26:33 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Sending snapshot: 1280 x 720 (cached) Doorbell false
[9/9/2021, 7:26:33 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Resize command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i pipe: -frames:v 1 -filter:v scale='min(1280,iw)':'min(720,ih)':force_original_aspect_ratio=decrease -f image2 - Doorbell false
[9/9/2021, 7:26:34 PM] [EufySecurity-1.0.0-rc.13] DEBUG: eufy HomeBase2 GET StationCurrentMode: { value: 63, timestamp: 1631179826634 }
[9/9/2021, 7:26:34 PM] [EufySecurity-1.0.0-rc.13] DEBUG: eufy HomeBase2 GET StationGuardMode: { value: 2, timestamp: 1630606846000 }
[9/9/2021, 7:26:34 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Doorbell Triggered GET EventSnapshotsActive: 0
[9/9/2021, 7:26:34 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Doorbell Triggered GET HomeKitCameraActive: 0
[9/9/2021, 7:26:34 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Doorbell GET DeviceMotionDetected: { value: false, timestamp: 0 }
[9/9/2021, 7:26:34 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Doorbell GET DeviceEnabled: { value: true, timestamp: 1630577055000 }
[9/9/2021, 7:26:34 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Doorbell GET DeviceMotionDetection: { value: true, timestamp: 1630577055000 }
[9/9/2021, 7:26:34 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot requested: 1280 x 720 Doorbell false
[9/9/2021, 7:26:34 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Sending snapshot: 1280 x 720 (cached) Doorbell false
[9/9/2021, 7:26:34 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Resize command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i pipe: -frames:v 1 -filter:v scale='min(1280,iw)':'min(720,ih)':force_original_aspect_ratio=decrease -f image2 - Doorbell false
[9/9/2021, 7:26:35 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot requested: 1280 x 720 Doorbell false
[9/9/2021, 7:26:35 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Sending snapshot: 1280 x 720 (cached) Doorbell false
[9/9/2021, 7:26:35 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Resize command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i pipe: -frames:v 1 -filter:v scale='min(1280,iw)':'min(720,ih)':force_original_aspect_ratio=decrease -f image2 - Doorbell false
[9/9/2021, 7:26:42 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Doorbell Property Changes: ringing { value: false, timestamp: 1631212002836 }
[9/9/2021, 7:26:42 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Doorbell DoorBell ringing
[9/9/2021, 7:26:42 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot requested: 1280 x 720 Doorbell false
[9/9/2021, 7:26:42 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i https://security-app-eu.eufylife.com/v1/XXXXXXXX -frames:v 1 -f image2 - -hide_banner -loglevel error Doorbell false
[9/9/2021, 7:26:42 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot requested: 1280 x 720 Doorbell false
[9/9/2021, 7:26:42 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot requested: 1280 x 720 Doorbell false
[9/9/2021, 7:26:42 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot requested: 1280 x 720 Doorbell false
[9/9/2021, 7:26:42 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot requested: 1280 x 720 Doorbell false
[9/9/2021, 7:26:42 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot requested: 1280 x 720 Doorbell false
[9/9/2021, 7:26:42 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot requested: 1280 x 720 Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Snapshot requested: 1280 x 720 Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Fetching snapshot took 0.541 seconds. Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Sending snapshot: 1280 x 720 Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Resize command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i pipe: -frames:v 1 -filter:v scale='min(1280,iw)':'min(720,ih)':force_original_aspect_ratio=decrease -f image2 - Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Sending snapshot: 1280 x 720 (cached) Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Resize command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i pipe: -frames:v 1 -filter:v scale='min(1280,iw)':'min(720,ih)':force_original_aspect_ratio=decrease -f image2 - Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Sending snapshot: 1280 x 720 (cached) Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Resize command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i pipe: -frames:v 1 -filter:v scale='min(1280,iw)':'min(720,ih)':force_original_aspect_ratio=decrease -f image2 - Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Sending snapshot: 1280 x 720 (cached) Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Resize command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i pipe: -frames:v 1 -filter:v scale='min(1280,iw)':'min(720,ih)':force_original_aspect_ratio=decrease -f image2 - Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Sending snapshot: 1280 x 720 (cached) Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Resize command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i pipe: -frames:v 1 -filter:v scale='min(1280,iw)':'min(720,ih)':force_original_aspect_ratio=decrease -f image2 - Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Sending snapshot: 1280 x 720 (cached) Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Resize command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i pipe: -frames:v 1 -filter:v scale='min(1280,iw)':'min(720,ih)':force_original_aspect_ratio=decrease -f image2 - Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Sending snapshot: 1280 x 720 (cached) Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Resize command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i pipe: -frames:v 1 -filter:v scale='min(1280,iw)':'min(720,ih)':force_original_aspect_ratio=decrease -f image2 - Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Sending snapshot: 1280 x 720 (cached) Doorbell false
[9/9/2021, 7:26:43 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Resize command: /homebridge/node_modules/homebridge-eufy-security/node_modules/ffmpeg-for-homebridge/ffmpeg -i pipe: -frames:v 1 -filter:v scale='min(1280,iw)':'min(720,ih)':force_original_aspect_ratio=decrease -f image2 - Doorbell false
[9/9/2021, 7:26:50 PM] [EufySecurity-1.0.0-rc.13] DEBUG: eufy HomeBase2 GET StationCurrentMode: { value: 63, timestamp: 1631179826634 }
[9/9/2021, 7:26:50 PM] [EufySecurity-1.0.0-rc.13] DEBUG: eufy HomeBase2 GET StationGuardMode: { value: 2, timestamp: 1630606846000 }
[9/9/2021, 7:26:50 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Doorbell Triggered GET EventSnapshotsActive: 0
[9/9/2021, 7:26:50 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Doorbell Triggered GET HomeKitCameraActive: 0
[9/9/2021, 7:26:50 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Doorbell GET DeviceMotionDetected: { value: false, timestamp: 0 }
[9/9/2021, 7:26:50 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Doorbell GET DeviceEnabled: { value: true, timestamp: 1630577055000 }
[9/9/2021, 7:26:50 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Doorbell GET DeviceMotionDetection: { value: true, timestamp: 1630577055000 }
github-actions[bot] commented 3 years ago

Did you check this Common Issues pages ?

lenoxys commented 3 years ago

Was able to reproduce. Working on this.

gielk commented 3 years ago

Awesome @lenoxys Hope u can fix the problem, I have exactly the same issue.

CooperCGN commented 2 years ago

Same problem here, first push message comes without snapshot, second then has one.

appstefan commented 2 years ago

Also experiencing this. Double notifications after one doorbell press. Sometimes they both include an image, more often just the second one does.

github-actions[bot] commented 2 years ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

gielk commented 2 years ago

@lenoxys Did u have any succes so far? The issue is still the same in the last version.

gielk commented 2 years ago

@lenoxys What is the status of this issue? At the moment, in v1.0.6 I still get 2 notifications when someone rings the doorbell. It's nice that it works, but annoying that you get 2 notifications everywhere.

Is there anything I can help with, do you need additional information, is there something I can try myself to solve it? My doorbell is model T8200.

rnldnkp commented 2 years ago

Same issue here, can't find out how to avoid the double notification.

Did notice you see two devices with the same identifier when you setup the plugin. Deselecting one of them does not fix it since the filter is filtering out 'both devices', result is no device at all. Must say I'm not even sure that's the way it's supposed to work ;)

lenoxys commented 2 years ago

plan for 1.1.0

github-actions[bot] commented 2 years ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

gielk commented 2 years ago

Still not fixed, unfortunately. I hope this is fixed soon because it is annoying 🥲 first world problem of course

lenoxys commented 2 years ago

Still not fixed, unfortunately. I hope this is fixed soon because it is annoying 🥲 first world problem of course

it's fixed on beta. I've a doorbell and had the issue on latest but fixed on beta.

gielk commented 2 years ago

Still not fixed, unfortunately. I hope this is fixed soon because it is annoying 🥲 first world problem of course

it's fixed on beta. I've a doorbell and had the issue on latest but fixed on beta.

How can I use the beta? Or is it due soon?

CooperCGN commented 2 years ago

With one of the earlier betas I had the problem that Ffmpeg kept crashing homebridge. Is this still the case?

schliemann commented 2 years ago

I can stream from the doorbell. But make sure to set enabledetailedlogging to at least 1.

github-actions[bot] commented 2 years ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.