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

Snapshot bei Bewegegunserkennung mit 404 #71

Closed
cratoo opened this issue Feb 1, 2021 · 10 comments
Closed

Snapshot bei Bewegegunserkennung mit 404 #71

cratoo opened this issue Feb 1, 2021 · 10 comments
Assignees
@bropat
Labels
bug Something isn't working fixed. waiting for feedback Issue should be fixed. Waiting on feedback

Comments

@cratoo
Copy link

cratoo commented Feb 1, 2021 •
edited

Describe the bug
Bei einer erkannten Bewegung versucht der Adapter den Snapshot abzuholen und bekommt einen 404.

To Reproduce
Steps to reproduce the behavior:

  1. Vor die Kamera treten.
  2. Push-Benachrichtigung abwarten
  3. Im Log nachsehen
  4. Fehler wie folgt:
error: eufy-security.0 (7675) saveImage(): getImage Error: Error: Request failed with status code 404 - url: https://security-app-eu.eufylife.com/v1/...
- info: eufy-security.0 (7675) state eufy-security.0.XXX.cameras.XXX.last_captured_pic_url changed: /eufy-security.0/last_captured_XXX.jpg (ack = true)

Expected behavior
Vermutlich ist der Adapter zu schnell und der Snapshot noch nicht verfügbar, denn beim nächsten Cloud Sync holt er ihn dann erfolgreich:

info: eufy-security.0 (7675) Refresh data from cloud and schedule next refresh.
...
info: eufy-security.0 (7675) state eufy-security.0.XXX.cameras.XXX.last_event_pic_html changed: <img src="data:image/jpg;base64,/9j/2wBDACA...

Versions:

  • Adapter version: 0.1.5
@bropat bropat added the bug Something isn't working label Feb 1, 2021
@bropat bropat self-assigned this Feb 1, 2021
@cratoo
Copy link
Author

cratoo commented Feb 5, 2021

Habe das jetzt noch mehrfach beobachtet:
Die Push-Nachrichten werden offenbar komplett erhalten, aber oftmals wird kein Bild abgeholt. Wäre es evtl. sinnvoll einfach nach ein paar Sekunden manuell ein Cloud-Sync zu triggern?
Hab in den Code reingeschaut und vielleicht würd' ich irgendwas hinbekommen, aber das wäre dann maximal ein "proof-of-concept" und sicher nichts was man behalten wollte...

@bropat bropat added the work in progress Working on it label Feb 7, 2021
@cratoo
Copy link
Author

cratoo commented Feb 8, 2021

@bropat wenn ich beim debuggen etc helfen kann, einfach sagen. :)

@cratoo cratoo mentioned this issue Feb 8, 2021
Closed
@bropat
Copy link
Owner

bropat commented Feb 8, 2021

@cratoo Ich habe bereits im aktuellen Entwicklungszweig ein Workaround implementiert, wo ich wenn ich ein HTTP Error 404 bekomme bis zu 5 mal erneut Versuche das Bild zu holen in wachsenden Zeitabstand (2. Versuch nach 5 Sek., 3. Versuch nach 10 Sek, usw.).
Ich werde diesen Fix mit dem nächsten Release veröffentlichen.

@cratoo
Copy link
Author

cratoo commented Feb 8, 2021

Top. Habe allerdings den Eindruck, dass da noch mehr sanity-checks notwendig werden. Teilweise bekomme ich Push-Nachrichten aufs Handy, aber der Adapter bekommt sie nicht?!
Frage mich, ob es eventuell an der Anzahl der Freigaben hängt? Also wieviele Geräte die Push-Nachricht bekommen sollen.

@bropat
Copy link
Owner

bropat commented Feb 8, 2021

aber der Adapter bekommt sie nicht?!

Wenn du den Adapter im Debug-Modus versetzt siehst du im Log keine Push-Benachrichtigung ankommen?

@cratoo
Copy link
Author

cratoo commented Feb 8, 2021

Ich hatte mich bisher auf das normale Log verlassen. Werde debug mal anschalten, ob im Falle von "ausgebliebenen" Push-Nachrichten der Adapter schlicht nichts bekommen hat oder was anderes schief läuft.

@bropat
Copy link
Owner

bropat commented Feb 8, 2021

Der Adapter ist noch stark in Entwicklung und somit ist es leicht möglich, dass ein oder der andere Fehler enthalten ist. Lasse mich Wissen ob der Adapter überhaupt keine Benachrichtigung erhält oder wenn ein Fehler im Adapter enthalten ist.

@cratoo
Copy link
Author

cratoo commented Feb 8, 2021

Dafür dass der Adapter "stark" in Entwicklung ist funktioniert er echt 1a! Und der Ausblick auf den Stream ohne die App ist großartig :)

@TbsJah
Copy link

TbsJah commented Feb 10, 2021

Ich bin auch Mega gespannt auf den Stream.
Wenn das funktioniert fliegt die Ring Klingel weg und wird durch Eufy ersetzt

bropat pushed a commit that referenced this issue Feb 14, 2021
Implemented P2P livestream over HLS
3a10dd3 
Last livestream is always saved and is still available later
Implemented device and station parameter refresh over P2P
Revised push notification implementation
Fixed issue #71 by implementing retry mechanism on HTTP error 404 (max. 5 retries with increasing delay)
Fixed issue #12
Eufy client library extracted as standalone library and adapters ported to new shared library: eufy-security-client
Removed following states: last_captured_pic_url, last_captured_pic_html
Updated versions of the package dependencies
@bropat bropat added fixed. waiting for feedback Issue should be fixed. Waiting on feedback and removed work in progress Working on it labels Feb 14, 2021
@cratoo
Copy link
Author

cratoo commented Feb 18, 2021

Mit der 0.2.3 sieht das für mich viel besser aus. Top und danke für den Fix.

@cratoo cratoo closed this as completed Feb 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bropat bropat

Labels
bug Something isn't working fixed. waiting for feedback Issue should be fixed. Waiting on feedback
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@cratoo @bropat @TbsJah