search

bropat / ioBroker.eusec

This ioBroker adapter allows to control Eufy security devices by connecting to the Eufy cloud servers.
MIT License
140 stars 18 forks source link

Snapshot bei Bewegegunserkennung mit 404 #71

Closed cratoo closed 3 years ago

cratoo commented 3 years ago

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:

cratoo commented 3 years ago

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...

cratoo commented 3 years ago

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

bropat commented 3 years ago

@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 commented 3 years ago

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 commented 3 years ago

aber der Adapter bekommt sie nicht?!

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

cratoo commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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

cratoo commented 3 years ago

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