-
Notifications
You must be signed in to change notification settings - Fork 171
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
Testing with Steve #37
Comments
Hi Victormunoz, Now that sounds like an interesting project. It may have no effect, but I can see you assign a client name to the json client, but also have it included in the url. |
related issue in our project: steve-community/steve#53 i also looked at this issue. there are two problems:
edit: per spec, "The exact OCPP version MUST be specified in the Sec-Websocket-Protocol field." edit2: you should have seen the exception in the logs of steve: "org.springframework.web.socket.server.HandshakeFailureException: No protocol (OCPP version) is specified." |
Now that's a bug. Let me check. |
i think, the problem lies in the OcppDraft. first of all, the methods on the other hand, you shouldn't set the you should only call its constructor by setting the wanted protocol, like:
but, i am not familiar with the code base and this is my observation after looking at the code for a few minutes. i hope it helps. edit: when you call the super constructor like that, you do not need to override these two methods anymore. |
Thanks for your input, and you're absolutely right. Luckily a debug showed that they don't get called. I reverted some PR some time ago that aimed to fix this, I'll reintroduce some of it when I have time. Thanks for your help, I really appreciate it. |
Not that I have anything to add re the "bug" at the moment, but SteVe does indeed look like an interesting project since apparently it supports OCPP 1.6J. It was searching for an open source 1.6J "server" that led me here in the first place. |
@V2G-UK you're not the only one. |
I have reintroduced the Sec-Websocket-Protocol part. Please check if it works. |
yes, the received heartbeat and the fact that there is one ocpp 1.6j charging station that is connected to steve are indicators that it is working. |
I have tested the updated code. |
the behaviour is correct. i fail to see where the problem is.
|
I should explain me better. But with this simulator, for some reason, the Boot Notification is not received in steve (therefore my code does not add the chargebox to the database, and so on), unless I add the simulator id to steve manually. |
are you using the ChargePoint in JSON mode? |
Yes. |
i cannot explain the exception with SOAP, @TVolden should look into it. in case of JSON, as I have said "steve can only accept or reject WebSocket connection attempts based on their id being in the database or not". the phrasing of this sentence was deliberate and not a mistake. the decision to accept a WebSocket connection attempt happens somewhere else in the code, here. a BootNotification can only arrive if the WebSocket connection is established. if you did not modify the part that i highlighted above, you should see exception like:
|
Victor, SteVe works fine for me when using WS/JSON as described by Sevket. If you modify SteVe then you're on your own! Unless Sevket graciously offers to help of course? Jim |
Thank you very much @goekay Regarding the exception with SOAP, I will investigate further before raising a new issue. |
I am trying to use this project as a chargepoint simulator to make tests with a central system that is running with Steve, now that it supports OCPP 1.6 (https://github.com/RWTH-i5-IDSG/steve).
But I haven't succeded on receiving any message in the Steve central system, neither using SOAP nor JSON types.
I have created a very basic main app that simply starts a chargepoint:
And in the constructor of the FakeChargePoint class I have changed the url to the ones that are provided by steve once it is started:
This code compiles and executes without problems or error messages. However, the Steve central system does not seem to be receiving anything at all from the chargepoint.
What is missing?
The text was updated successfully, but these errors were encountered: