Need better checks when refreshing credentials

[ivanfmartinez]

Describe the bug
I have received a new email from volvo with an OTP token, when I checked the logs of volvo2mqtt I found that was an error during refresh credentials.

In my case the server returned 504 error, which does not mean that credentials are invalid, just does not was able to fullfill request.

I have stopped and restarted the container without doing the OTP and the credentials on token_file are still valid and were refreshed when container is restarted.

To Reproduce:

Unable to manually reproduce

Expected behavior:

In some cases the refresh should be retried later without requesting a new login

I dont know what errors should be considered, but for now I think that 5xx can be considered as temporary error like the RequestException is considered.

I have added the 5xx check to my existing PR : #267

Version info:

docker v1.10.5

Logs:

volvo2mqtt    | Nov 06 07:39:36 volvo2mqtt [MainThread] [1] - INFO: Sending mqtt update...
volvo2mqtt    | Nov 06 07:39:36 volvo2mqtt [MainThread] [1] - INFO: Refreshing credentials
volvo2mqtt    | Nov 06 07:39:51 volvo2mqtt [MainThread] [1] - WARNING: Refreshing credentials failed!: 504 Message: upstream request timeout
volvo2mqtt    | Nov 06 07:39:51 volvo2mqtt [MainThread] [1] - INFO: Starting login with OTP

[github-actions]

This issue is stale because it has been open 15 days with no activity. Remove stale label or comment or this will be closed in 5 days.

[github-actions]

This issue was closed because it has been stalled for 5 days with no activity.