cliviu74 / wallbox

Python Module interface for Wallbox EV chargers api
Apache License 2.0
60 stars 21 forks source link

Divide jwtTokenTtl by 1000 to compensate for milisecond timestamp #19

Closed hesselonline closed 1 year ago

hesselonline commented 1 year ago

The Ttl timestamp from the API seems to have changed in formatting and now has 13 characters (includes miliseconds). This results in this comparison not working anymore. Which in turn causes errors downstream (Home Assistant integration broken).

hesselonline commented 1 year ago

Hi @cliviu74 I'm the maintainer of the Home Assistant Wallbox integration; we use your Pypi package for connectivity to the wallbox API. I believe something changed in the api ttl timestamp, this is currently causing issue with home assistant. (https://github.com/home-assistant/core/issues/79785). Hope you have time to merge this fix

cliviu74 commented 1 year ago

LGTM, thanks for the fix @hesselonline.

cliviu74 commented 1 year ago

@hesselonline fix released to module version 0.4.10