openhab / openhab-addons

Add-ons for openHAB
https://www.openhab.org/
Eclipse Public License 2.0
1.86k stars 3.58k forks source link

[hydrawise] Communication error #16570

Closed clown-m closed 2 weeks ago

clown-m commented 5 months ago

I upgraded to 4.1.2 today while i saw that there is an authorization bugfix in the hydrawise binding done, but still i got this message after the upgrade in the BASIC UI Thing setup:

COMMUNICATION_ERROR
Request failed with HTTP status code: 400 response: {"errors":{"message":"Could not resolve promise"}}_

I am running the Hydrawise Binding on openHAB 4.1.2 stable version on a Debian Buster Machine.

This failure comes up since some days, all other APP's are running as normal.

Any idea?

DenDeXTeR51 commented 1 month ago

UPD: I try ro instal from Addon Store + "bundle:update org.openhab.binding.hydrawise https://github.com/digitaldan/openhab-addons/releases/download/hydrawise-weather-fix/org.openhab.binding.hydrawise-4.2.0-SNAPSHOT.jar" in both cases same error :

Request failed with HTTP status code: 400 response: {"errors":{"message":"Could not resolve promise"}}

digitaldan commented 1 month ago

in the openhab console , what does list -s |grep hydrawise output ?

DenDeXTeR51 commented 1 month ago

@digitaldan
368 │ Active │ 80 │ 4.2.0.202404072314 │ org.openhab.binding.hydrawise

digitaldan commented 1 month ago

Let me post a new version today, there's a few more changes that have not been published yet.

DenDeXTeR51 commented 1 month ago

@digitaldan Ping me in reply when you will upload new version. Thanks!

digitaldan commented 1 month ago

Give this one a try, it should attempt to try weather once, and if it fails, it will continue on without it and not try again. The jar says version 4.3, but it should work fine on 4.2 systems

https://github.com/digitaldan/openhab-addons/releases/download/hydrawise-weather-fix/org.openhab.binding.hydrawise-4.3.0-SNAPSHOT.jar

DenDeXTeR51 commented 1 month ago

@digitaldan It works!

digitaldan commented 1 month ago

Excellent, thanks for testing, i'll open a PR up and get this into the main repo.

digitaldan commented 1 month ago

Actually, is weather working now for you ? I changed it so it requests weather data as a separate call, was thinking this might actually work around their issue .

DenDeXTeR51 commented 1 month ago

Seems forecast working too