search

audiconnect / audi_connect_ha

Adds an audi connect integration to home assistant
MIT License
227 stars 98 forks source link

fix: Critical Fix - Trip Data 403 #388

Closed coreywillwhat closed 5 months ago

coreywillwhat commented 5 months ago

going back to #254 it looks like no safeguard was in place for get_trip_data and this was part of the reason this was disabled. This means repeated 403 responses with every update if a vehicle isn't reporting trip data. This PR adds a check similar to the other data functions which will check once, and upon 403 will disable support for the service and wont check again until next HA restart.

coreywillwhat commented 5 months ago

@Kolbi or @cdnninja if you have a few mins, would like to merge this as soon as possible, as well as manually push a new version

Kolbi commented 5 months ago

Can test it in around 30 minutes and do the rest.

Today I had a view in the Audi App with an Q4 etron - the App looks different on that vehicle.

Kolbi commented 5 months ago

Tested & working for me.

Let's see on other vehicles :)

coreywillwhat commented 5 months ago

Awesome thank you!