Mercedes: refactor token refresh

[andig]

Replace #18338

[ReneNulschDE]

@andig : kann wenig nachvollziehen warum du das extra in ein anderen PR geworfen hast und nicht die Diskussion in meinem PR angestossen hast oder zumindest wartest, bis ich aus dem WE zurück bin... Das hier funktioniert bei mir nicht.

Du musst die DB Token zuerst nehmen... Hab da mehrere Stunden drauf verbraucht um einen funktionierenden Weg zu finden.

[andig]

kann wenig nachvollziehen

Für mich war es viel einfacher vom bestehenden Code auszugehen der ja prinzipiell funktioniert. Aber sei's drum- wenn das ein Problem ist kann ich den auch reverten.

Hab da mehrere Stunden drauf verbraucht um einen funktionierenden Weg zu finden.

Das sieht man dem PR tatsächlich auch an- er wirkt sehr kompliziert. Ich finde es immer gut, Ursache und Wirkung zu verstehen.

Du musst die DB Token zuerst nehmen

Die beiden lassen sich sicher umdrehen. Kannst Du erklären warum? Wenn der aus der Config nicht geht wird dieser ja letztlich genommen?

[ReneNulschDE]

Erklären kann ich es nicht, aber was habe ich gesehen:

1. Der Config RT Token ist mehrfach gültig in den ersten 4 Stunden.
   1a. Wird also evcc in den ersten 4 Stunden neu gestartet geht es mit dem Config RT (und der neu token wird in der DB aktualisiert)
   1b. Wird evcc nach 4 Stunden neu gestartet, schlägt der Versuch mit Config RT fehl und der DB RT wird auch ungültig
2. Dreht man die Reihenfolge um und nutzt erst den DB Token funktioniert es in beiden Fällen, Da mit dem DB RT noch kein neues Tokenset abgerufen wurde.

Meine Vermutung, wenn ein neuer RT ausgestellt wurde und mit einem alten RT dessen AT abgelaufen ist, wieder ein Abruf erfolgt, werden alle RTs in der Kette ungültig.

[andig]

Moin @ReneNulschDE wenn das so ist habe ich den Faden verloren. Ziel Deines PRs war doch

try to use the evcc.yaml based token before checking the settings store token.

Wenn das aber eh nicht geht- was/warum wollten wir überhaupt ändern? Den hier reverten?

[ReneNulschDE]

Du musst deinen Code so ändern, dass der DB Token vor dem Config Token geladen wird. Dann sollte es gehen.

[andig]

Das war ja aber vorher auch so. Also warum sollten wir überhaupt etwas ändern? Was war denn dann Ziel des PRs???

[ReneNulschDE]

Das war ja aber vorher auch so. Also warum sollten wir überhaupt etwas ändern? Was war denn dann Ziel des PRs???

??? Du wolltest dass man einen neuen Token in die Config einfügen kann und der genutzt werden kann, wenn der DB Token ungültig ist.

Aber die gesamte Anmeldung geht seit heute morgen nicht mehr, da MB den PIN-Auth Mode abgeschalten hat. Ich arbeite jetzt erstmal an der HA Integration. Die Erkenntnisse kannst Du dann später nutzen...