Ocpp: make sure central system always responds to chargepoint requests #16279
Conversation
| return cp.OnBootNotification(request) | ||
| res := &core.BootNotificationConfirmation{ | ||
| CurrentTime: types.Now(), | ||
| Interval: 60, |
There was a problem hiding this comment.
Das ist in dem Fall wahrscheinlich etwas lang.
hier auch Timeout verwenden?
Ist ja dann das Retry-Intervall des CP.
There was a problem hiding this comment.
Eigentlich sollte der PR keine Funktion ändern- das ist schon immer so. Können wir aber gerne separat anpassen.
There was a problem hiding this comment.
Vorher wurde aber nie mit Pending sondern immer mit Accepted (am CP) geantwortet.
Das ist ein entscheidender Unterschied in der Bedeutung von Interval.
There was a problem hiding this comment.
Die Pending Antwort ist jetzt komplett neu. Ich habs im master auf Timeout geändert (genauso wie accepted). Vorher gabs da gar nichts.
There was a problem hiding this comment.
Accepted sollte wie vorher (am LP) auf 60s bleiben, da sonst manche Boxen mit dem Hearbeat-Intervall < 60s Probleme haben.
Im Fall von Pending ist die Bedeutung von Interval: aber eben nicht "Hearbeat-Interval" sondern "Retry-Interval". Also etwas anderes.
Da ist Timeout sinnvoll.
Refs #15677. Follow-up to #15950.