Update pycarwings2.py #2842
Update pycarwings2.py #2842
Conversation
Two changes
1st re line 75: Nissan changed the Base URL of its Carwings API. Since May 20, 2024 the old URL used by pycarwings2 in soc_leaf doesn't work anymore.
2nd re line 121: At least on a Windows 10 PC with Python 3.12 for test purpose, the additional parameter 'headers={"User-Agent": ""}' is needed to run pycarwings2.py successfully. The need for this parameter within OpenWB itself has tbc.
For more details see posts on https://forum.openwb.de/viewtopic.php?p=109761#p109761 ff
|
Wenn ich den Beitrag im Forum und Deine Beschreibung richtig gelesen habe, wurde diese Anpassung noch nicht auf einer openWB 1.9 mit Stretch getestet, sondern nur unter Windows mit Python 3.12? Dann diesen Test bitte noch nachholen. |
benderl
added
the
waiting for feedback
|
Das würde ich zwar gerne machen, ich habe aber weder einen Pi (außer dem Pi in der OpenWB) noch sonstige Erfahrungen mit einem Pi. Ich müsste hier bei nahe null anfangen.
Die Änderung der URL auf die Nissan API ist in jedem Fall erforderlich.
Die EVCC Kollegen hatten das gleiche Problem mit dem Verlust des Zugriffs auf die Nissan API, siehe evcc-io/evcc#13941
Sie haben die neue URL https://gdcportalgw.its-mo.com/api_v230317_NE/gdc/ als Lösung gefunden.
Der Autor der My Leaf App (Tobias Kjeldsen) hat den Link am 22.05. in seinem Code ebenfalls angepasst, siehe Zeile 20 in
https://github.com/Tobiaswk/carwingsflutter/blob/master/android/app/src/main/java/dk/kjeldsen/carwingsflutter/CarwingsSession.java .
Er nutzt Carwings zwar auf der Java Plattform. Aber das ist bei einer Basis ULR nicht relevant.
Etwas anderes ist es mit dem zusätzlichen Parameter beim Aufbau der
HTTPS Session. Es ist gut möglich, dass der nur auf einem PC oder nur
mit einer neuen Python Version nötig ist (ich nutze 3.12).
Der zusätzliche Parameter sollte also erst auf einem Pi probiert werden, was
ich bislang aber nicht kann.
|
|
Bitte im Forum nach jemandem fragen, der die Anpassung testen und auch beurteilen kann, ob der Header notwendig ist. |
|
Ich werde den Vorschlag mit dem Aufruf im Forum probieren. Parallel schaue ich gerade, ob ich aus meinem Bekanntenkreis einen Pi ausleihen kann. Dort würde ich mein zum Hauptprogramm mutiertes SoC_Leaf.py zusammen mit responses.py und dem pycarwings2.py mit den hier genannten Änderungen testen (mal mit dem zusätzlichen Parameter im HTTPS-Aufbau, mal ohne) |
|
Ich habe mit zwei Versionen von modifizierten pycarwings2.py erfolgreich getestet. Variante 2: Umgebung: b) c) sudo python /var/www/html/openWB/modules/soc_leaf/soc.py $username $password $CHARGEPOINT d) Ein Auszug aus /var/www/html/openWB/ramdisk/soc.log schaut gut aus und weist keine Fehler auf: |
|
Das ging schnell! Super, vielen Dank! |
|
Bitte keinen neuen PR, damit die Beiträge hier auch erhalten bleiben. Füge Deinem Branch einfach noch einen zweiten Commit hinzu, der die Änderung am Haeder rückgängig macht. |
2nd change removed, because it is not needed on the OpenWB software platform.
|
Erledigt, jetzt steht in meinem Code nur noch das URL Update |
|
Kann ich noch etwas tun? |
|
Der Fehler ist unabhängig von Deinem PR. Sieht so aus, als wäre Python 3.5 nun endgültig End-Of-Life. |
|
Super, vielen Dank für den Merge |
Two changes
1st re line 75: Nissan changed the Base URL of its Carwings API. Since May 20, 2024 the old URL used by pycarwings2 in soc_leaf doesn't work anymore.
2nd re line 121: At least on a Windows 10 PC with Python 3.12 for test purpose, the additional parameter 'headers={"User-Agent": ""}' is needed to run pycarwings2.py successfully. The need for this parameter within OpenWB itself has tbc.
For more details see posts on https://forum.openwb.de/viewtopic.php?p=109761#p109761 ff