search

nkgilley / python-ecobee-api

Python API for controlling Ecobee Thermostats
MIT License
44 stars 40 forks source link

Refresh tokens on 500 status code #44

Closed marthoc closed 4 years ago

marthoc commented 4 years ago

Testing in Home Assistant has revealed that the ecobee API can also throw a 500 status code when the access token needs refreshing. This PR catches 500, and also improves the error log messages to aid in diagnosing problems in the future.

marthoc commented 4 years ago

@nkgilley If you could merge this and then push 0.1.4 to PyPI, I should be able to get this included in Home Assistant for the 0.100 release. Thanks!

nkgilley commented 4 years ago

Done. Thanks!

On Mon, Sep 30, 2019 at 11:32 AM Mark Coombes notifications@github.com wrote:

@nkgilley https://github.com/nkgilley If you could merge this and then push 0.1.4 to PyPI, I should be able to get this included in Home Assistant for the 0.100 release. Thanks!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/nkgilley/python-ecobee-api/pull/44?email_source=notifications&email_token=AALQBKXCHE2U3YQY5352F23QMIL2RA5CNFSM4I3VMTQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD76CGCQ#issuecomment-536617738, or mute the thread https://github.com/notifications/unsubscribe-auth/AALQBKXEO5QOK6SERMEJFWTQMIL2RANCNFSM4I3VMTQA .