GusMuche / homebridge-zipabox-platform

Homebridge plugin to Link HomeKit and the Zipabox through API request. Implementation in dynamic platform.
MIT License
5 stars 2 forks source link

Invalid jsonresponse #25

Closed thierry13790 closed 4 years ago

thierry13790 commented 4 years ago

Bonjour J’ai cette erreur dans les logs même avec la dernière version Merci d’avance Thierry

263152AD-CAD4-4D86-B9F9-FF70DEB0C3DF

GusMuche commented 4 years ago

Besoin de plus d’info :

A ce stade je vois une erreur dans le retour de la box, une alternative est de voir si la requête en manuel passe

thierry13790 commented 4 years ago

C'est en remote, pas d'autres erreurs avec d'autres accessoires, et du coup je pense avoir trouvé le pb en comparant avec d'autres capteurs de T°, l'erreur est lié au parametrage de status

Ma config :

image

Si je passe noStatus=false

image

J'ai d'autres capteurs de T° avec pas d'erreur mais ceux cis sont des capteurs de temperature virtuels alors que celui est directement connecté sur la zipabox c'est un Oregon scientific et il a plusieurs valeurs possibles : humidité ou Temperature ceci explique peut etre le probleme. A noter egalement que ce capteur fonctionne malgré l'erreur dont on parle via homebridge la bonne Temperature s'affiche.

GusMuche commented 4 years ago

Vérifie bien le UUID fourni. Il doit être celui de l’attribut Temperature du plus bas niveau.

Possible de tester via une requête API depuis l’interface de Zipato en communiquant la valeur dans la requête attributes/{UUID}/statuses (de mémoire)

thierry13790 commented 4 years ago

Ok je vais contrôler ça merci de l’info

GusMuche commented 4 years ago

Il existe aussi une requête API qui liste tous les status dispo. Je pense coder cela afin que le plugin prévienne si le UUID fourni n’est pas dans la liste. Ca évitera l’apparition d’erreurs liées à cela.

thierry13790 commented 4 years ago

Bonne idée

Le 22 mai 2020 à 11:15, Gus Muche notifications@github.com a écrit :

 Il existe aussi une requête API qui liste tous les status dispo. Je pense coder cela afin que le plugin prévienne si le UUID fourni n’est pas dans la liste. Ca évitera l’apparition d’erreurs liées à cela.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

GusMuche commented 4 years ago

Peux-tu svp tester avec la nouvelle version et me dire si le problème est toujours présent ?

thierry13790 commented 4 years ago

Merci oui je reste vite

Thierry

Devise ''on a toujours du temps pour faire ce que l'on a envie de faire...''

Le 6 juin 2020 à 12:17, Gus Muche notifications@github.com a écrit :

 Peux-tu svp tester avec la nouvelle version et me dire si le problème est toujours présent ?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

thierry13790 commented 4 years ago

C'est installé je te tiens au courant le problème n'apparaissait pas de suite j'attend quelques jours et te donne le status !

GusMuche commented 4 years ago

Des nouvelles de cette erreur ?

thierry13790 commented 4 years ago

Bonjour

Le problème est résolu merci Gusmuche !

Thierry

Le 3 juil. 2020 à 13:14, Gus Muche notifications@github.com a écrit :

 Des nouvelles de cette erreur ?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.