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

Wiederholender Ladeplan: request failed with status code 400 #17850

Closed
1 task done
An-Wi opened this issue Dec 21, 2024 · 10 comments · Fixed by #17885
Closed
1 task done

Wiederholender Ladeplan: request failed with status code 400 #17850

An-Wi opened this issue Dec 21, 2024 · 10 comments · Fixed by #17885
Assignees
Labels
ux User experience/ interface

Comments

@An-Wi
Copy link

An-Wi commented Dec 21, 2024

Describe the bug

Beim Versuch, einen wiederholenden Plan hinzuzufügen, passiert zunächst „nichts“. In den Meldungen ist dann folgendes zu finden:
IMG_4031

Steps to reproduce

  1. Ladeplanung
  2. Wiederholenden Plan hinzufügen
  3. Meldungen einsehen

Configuration details

.

Log details

(keine dazu passenden Einträge)

What type of operating system are you running?

Windows

Nightly build

  • I have verified that the issue is reproducible with the latest nightly build

Version

0.131.12

@andig andig added the ux User experience/ interface label Dec 22, 2024
@andig
Copy link
Member

andig commented Dec 22, 2024

Kannst Du das reproduzieren? Wie sieht der Request aus (Entwicklerkonsole des Browsers, Netzwerktab)?

@naltatis
Copy link
Member

Ok, das ist merkwürdig. Wir ermitteln in der Tat die aktuelle Zeitzone für den wiederholenden Plan im Browser und nutzen auch genau die in Go um dann die Plan auszuwerten. Letzteres scheint hier ja fehlzuschlagen obwohl "Europe/Rome" eine valide Zeitzone ist. https://en.wikipedia.org/wiki/List_of_tz_database_time_zones
Bist du in Italien unterwegs? Bei mir ists "Europe/Berlin"

@An-Wi
Copy link
Author

An-Wi commented Dec 22, 2024

Screenshot oben ist vom iPhone.
Wenn ich es auf dem iPad oder am PC (Google Chrome) teste, kommt die gleiche Fehlermeldung, aber mit „unknown time zone Europe/Berlin.“

@andig
Copy link
Member

andig commented Dec 22, 2024

#17850 (comment)

?

@An-Wi
Copy link
Author

An-Wi commented Dec 23, 2024

Kannst Du das reproduzieren? Wie sieht der Request aus (Entwicklerkonsole des Browsers, Netzwerktab)?

Console:
ScreenShot_9741_

Network:
ScreenShot_9736_
ScreenShot_9737_
ScreenShot_9738_
ScreenShot_9739_
ScreenShot_9740_

@naltatis
Copy link
Member

Hm, könnte das ein Go/Windows Problem sein?
golang/go#62065

@naltatis
Copy link
Member

@andig
Copy link
Member

andig commented Dec 23, 2024

Eine Lösung wäre anscheinend die TZ Informationen in evcc einzubauen (https://stackoverflow.com/questions/54363451/setting-timezone-globally-in-golang). Europe/Berlin ist m.E. aber eine Location, keine Zeitzone. Kann es sein, dass das im Code für den falschen Zweck verwendet wird? Siehe auch Link oben mit Codebeispiel.
Option: immer UTC für die APIs verwenden?

@naltatis
Copy link
Member

naltatis commented Dec 24, 2024

Option: immer UTC für die APIs verwenden?

Das funktioniert leider nicht. Wir speichern ja keinen festen Zeitpunkt ab sondern Wochentage + Uhrzeit. Um daraus einen Zeitpunkt errechnen zu können müssen wir die Zeitzone/Location des Anwenders kennen. Sonst verschiebt sich zur Sommer/Winterzeit der Ladeplan.

@naltatis
Copy link
Member

naltatis commented Dec 24, 2024

Die pun Implementierung nutzt den TZ Mechanismus auch bereits. Das müssten auf Windows heute auch bereits nen Fehler geben. Vermutlich nutzt die Kombination niemand :D
https://github.com/evcc-io/evcc/blob/master/tariff/pun.go#L184

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ux User experience/ interface
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants