Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update pycarwings2.py with header "User-Agent" #2864

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

mekrapp
Copy link
Contributor

@mekrapp mekrapp commented Jan 28, 2025

In der OpenWB V1.9.304N, im Modul pycarwings2 zur SoC Abfrage des Nissan Leaf muss der Aufruf der Nissan Carwings API in Zeile 121 um den zusätzlichen Parameter headers= {"User-Agent": ""} ergänzt werden.

Ohne diesen Parameter antwortet die API seit einigen Tagen mit dem Fehler INVALID PARAMS.

Der zusätzliche Parameter stammt aus der pycarwings2 Version von filcole https://github.com/filcole/pycarwings2/blob/master/pycarwings2/pycarwings2.py in Zeile 125.

Auf einem PC läuft die SoC-Abfrage mit pycarwings2 nur mit diesem zusätzlichen Parameter. Ansonsten kommt o.g. Fehlermeldung.

Der Parameter wurde bereits im PR #2842 von @ddraeyer getestet, dann aber mangels Notwendigkeit doch nicht eingeführt. Inzwischen kommt die o.g. Fehlermeldung aber auch bei Aufruf der API mit der OpenWB V1.9.304N.

In der OpenWB V1.9.304N, im Modul pycarwings2 zur SoC Abfrage des Nissan Leaf muss der Aufruf der Nissan Carwings API in Zeile 121 um den zusätzlichen Parameter headers= {"User-Agent": ""} ergänzt werden. 

Ohne diesen Parameter antwortet die API seit einigen Tagen mit dem Fehler INVALID PARAMS. 

Der zusätzliche Parameter stammt aus der pycarwings2 Version von filcole https://github.com/filcole/pycarwings2/blob/master/pycarwings2/pycarwings2.py in Zeile 125. 

Auf einem PC läuft die SoC-Abfrage mit pycarwings2 nur mit diesem zusätzlichen Parameter. Ansonsten kommt o.g. Fehlermeldung. 

Der Parameter wurde bereits im PR snaptec#2842 von @ddraeyer getestet, dann aber mangels Notwendigkeit doch nicht eingeführt. Inzwischen kommt die o.g. Fehlermeldung aber auch bei Aufruf der API mit der OpenWB V1.9.304N.
@benderl benderl merged commit 7285172 into snaptec:master Jan 30, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants