Ocpp: async RemoteStartTransaction by StatusNotification

[premultiply]

Fixes #15857
Fixes #15552 (comment)

[premultiply]

@andig Bitte mal nochmal kritisch auf Locking etc. schauen.

Funktioniert zumindest im lokalen Test damit nun wie erwartet - und schnell.

[andig]

Hab ich ganz bewusst so umgebaut.

Sollten wir trotzdem ändern. Es ist schlechte Kapselung, in eine Komponente rein zu greifen um Daten wieder an die gleiche Komponente zu übergeben. Das ist Spaghetticode ;)

[andig]

Wäre klasse das eine defer noch zu entsorgen falls möglich.

[premultiply]

Wird gebraucht um erst die Bestätigung der StatusNotification zu senden und dann erst RemoteStartTransaction auszuführen.
Andernfalls währe das verschachtelt und synchron und mit Deadlock-Risiko.

[andig]

Verstehe ich nicht ;) Gehts oder gehts nicht? Wir sollten nicht raten... Ein CP muss immer asynchron arbeiten können. Ich sehe kein Deadlockrisiko ausser in der Library.

[premultiply]

Muss ist so ein unpassendes Wort bei OCPP wie wir gelernt haben. ;)
Also Risikominimierung und definitiv übersichtlichere Logs.

[andig]

Der Grund, dass das Log hier unübersichtlich ist liegt einfach daran, dass die Funktion auf den Roundtrip wartet. Die Lösung wäre nur das Start zu schicken und nicht auf die Antwort zu warten. Dass ist eh sinnlos da außer Logging nichts damit passiert, insbesondere ja auch kein Retry.

Was mir noch auffällt: falls das scheitert gibt es beim Enable() keinen erneuten Versuch eine Transaktion zu starten. Das ist hier also einmal oder nicht.

[andig]

Noch eine Anmerkung zum defer: das defer passiert nach Funktions-Body, aber vor Return. Mit anderen Worten: auch mit defer ändert sich gar nichts daran, dass die innere Nachricht vollständig abgearbeitet wird bevor die äußere abgeschlossen wird. Es wäre super, hier auf konkrete Ergebnisse aufzusetzen statt auf Mutmaßungen.

[premultiply]

Ich kann mich nur daran halten was ich im Log sehe.
Da funktioniert es nur mit defer wie erwartet.

[premultiply]

Wenn das nicht reicht müsste man irgendwie völlig asynchron starten.