Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Push notification for doorbell ring does not update snapshot #86

Closed
cratoo opened this issue Feb 18, 2021 · 1 comment
Closed

Push notification for doorbell ring does not update snapshot #86

cratoo opened this issue Feb 18, 2021 · 1 comment
Assignees
Labels
bug Something isn't working fixed

Comments

@cratoo
Copy link

cratoo commented Feb 18, 2021

After the fix for motion detected snapshots, apparently something is wrong with doorbell ring snapshots. If someone rings, the adapter receives the push notification, but the snapshot is apparently not downloaded and the states are also not updated.

See the log, directly with silly logging :)

2021-02-18 16:56:11.575  - ESC[34mdebugESC[39m: eufy-security.0 (2760) PushClient.handleParsedMessage(): DataMessageStanza: message: {"tag":8,"object": 
{"id":"XXX","from":"348804314802","to":"XXX","category":"com.oceanwing.battery.cam","appData":[{"key":"content","value":"Doorbell:Someone has press doorbell"},{"key":"device_sn","value":"T82XXX"},{"key":"event_time","value":"1613663771068"},{"key":"payload","value":"XXX"},{"key":"push_time","value":"1613663771529"},{"key":"station_sn","value":"T8XXX"},{"key":"title","value":""},{"key":"type","value":"7"},{"key":"google.c.sender.id","value":"34XXX"}],"persistentId":"0:1613663771547269%0d2a775cf9fd7ecd","ttl":3600,"sent":"1613663771538"}}

2021-02-18 16:56:11.575  - ESC[35msillyESC[39m: eufy-security.0 (2760) PushNotificationService.onMessage(): raw_message: [{"id":"XXX","from":"348804314802","to":"edoHbP3qEPnhXXX","category":"com.oceanwing.battery.cam","persistentId":"0:1613663771547269%0d2a775cf9fd7ecd","ttl":3600,"sent":"1613663771538","payload":{"content":"Doorbell:Someone has press doorbell","device_sn":"T821XXX","event_time":"1613663771068","payload":{"event_type":3103,"device_sn":"T82XXX","name":"Doorbell","channel":0,"cipher":42,"session_id":"20210218_165611","pic_url":"https://security-app-eu.eufylife.com/v1/s/g/XXX","create_time":1613663771068,"file_path":"h265_20210218165610","notification_style":3,"push_count":1},"push_time":"1613663771529","station_sn":"T80XXX","title":"","type":"7","google.c.sender.id":"348XXX"}}]

2021-02-18 16:56:11.576  - ESC[35msillyESC[39m: eufy-security.0 (2760) PushNotificationService.onMessage(): message: [{"name":"Doorbell","event_time":1613663771068,"type":7,"station_sn":"T8XXX","device_sn":"T82XXX","title":"","content":"Doorbell:Someone has press doorbell","push_time":1613663771529,"channel":0,"cipher":42,"event_session":"20210218_165611","event_type":3103,"file_path":"","pic_url":"https://security-app-eu.eufylife.com/v1/s/g/XXX","push_count":1,"notification_style":3}]

2021-02-18 16:56:11.576  - ESC[34mdebugESC[39m: eufy-security.0 (2760) handlePushNotifications(): push_msg: {"name":"Doorbell","event_time":1613663771068,"type":7,"station_sn":"T80XXX","device_sn":"T82XXX","title":"","content":"Doorbell:Someone has press doorbell","push_time":1613663771529,"channel":0,"cipher":42,"event_session":"20210218_165611","event_type":3103,"file_path":"","pic_url":"https://security-app-eu.eufylife.com/v1/s/g/XXX","push_count":1,"notification_style":3}
@bropat bropat self-assigned this Feb 20, 2021
@bropat bropat added bug Something isn't working work in progress Working on it labels Feb 20, 2021
bropat pushed a commit that referenced this issue Feb 20, 2021
Fixed not correctly identifying if the livestream is still active or not
@bropat bropat added fixed. waiting for feedback Issue should be fixed. Waiting on feedback and removed work in progress Working on it labels Feb 20, 2021
@sunsear
Copy link

sunsear commented Feb 20, 2021

I can confirm that this did NOT work in 0.2.2 and does now work in 0.2.4. AFAI can tell, bug fixed. Motion detected, person detected and ringing detected are all working properly now. 👏

@bropat bropat added fixed and removed fixed. waiting for feedback Issue should be fixed. Waiting on feedback labels Feb 20, 2021
@bropat bropat closed this as completed Feb 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed
Projects
None yet
Development

No branches or pull requests

3 participants