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.

Sign up for GitHub

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

Jump to bottom

[Bug]: Prefetch from two Video Doorbells 2K #81

Closed
TOTO-DK opened this issue Jun 17, 2022 · 7 comments · Fixed by #154
Closed

[Bug]: Prefetch from two Video Doorbells 2K #81

TOTO-DK opened this issue Jun 17, 2022 · 7 comments · Fixed by #154
Labels
bug Something isn't working fixed in beta / next release this is a resolved issue which is fixed in current beta and will be fixed in next release

Comments

@TOTO-DK
Copy link

TOTO-DK commented Jun 17, 2022 •
edited
Loading

Deleted after examination
@TOTO-DK TOTO-DK added the bug Something isn't working label Jun 17, 2022
@github-actions
Copy link

Did you check this Common Issues pages ?

@thieren
Copy link
Contributor

thieren commented Jun 17, 2022

Hey @TOTO-DK

are these two devices of the same kind/type? Maybe post the first 5-6 characters of the serials to let me know.

Also I'd need the debug log to get a better view of what's happening.
You can activate it in the advanced section of the config-ui.

But a normal (local) livestream with the backdoor camera works? What about a snapshot from events or when you're in the HomeKit preview? (Actually works quite the same, so would baffle me if these work).

thieren added a commit to thieren/Homebridge-eufy-plugin that referenced this issue Jun 17, 2022
@thieren
changed: automatic refresh for multiple cameras should start at diffe…
736855b 
…rent times now (at 1 minute intervals)

fixed: release refresh lock if no camera source could be retrieved (see homebridge-eufy-security#81)
This was referenced Jun 17, 2022
Merged
Closed
@thieren
Copy link
Contributor

thieren commented Jun 17, 2022

I think this should be fixed in beta.18 - please try and let me know.

As to why this happened there are multiple causes:

  1. it seems that one station can only handle 1 concurrent p2p stream. So that when both try to access their streams it fail for the device that came last. Since the automatic refreshes started at the same time one was doomed to get no stream.
    If this is generally the case, or if this is a bug in eufy-security-client, I don't know as of yet. See here
  2. There was a bug that when no stream could be established the refresh process was locked and didn't get unlocked. This I did fix with the new beta, hopefully.

Thanks for the logs. You may want to delete any links to your cloud snapshots in the future, since these can be viewed by anyone.

@TOTO-DK
Copy link
Author

TOTO-DK commented Jun 17, 2022

Beta 18 WORKS - great job ;-)

@TOTO-DK TOTO-DK closed this as completed Jun 17, 2022
@thieren
Copy link
Contributor

thieren commented Jun 17, 2022

Can you send me a log with 5 and 6 minutes refreshes nonetheless?
I couldn't test myself and this way there should be a overlap after the first 6 minutes. I just want to make sure it's doing it right.

@thieren thieren added the fixed in beta / next release this is a resolved issue which is fixed in current beta and will be fixed in next release label Jun 17, 2022
@TOTO-DK
Copy link
Author

TOTO-DK commented Jun 17, 2022 via email

@thieren thieren mentioned this issue Sep 3, 2022
Merged
@TOTO-DK
Copy link
Author

TOTO-DK commented Oct 11, 2022 via email

@lenoxys lenoxys mentioned this issue Oct 23, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working fixed in beta / next release this is a resolved issue which is fixed in current beta and will be fixed in next release
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

2.1.0 thieren/Homebridge-eufy-plugin
2 participants
@thieren @TOTO-DK