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

Deviation from OCPP specification #37

Open
agruenb opened this issue Feb 25, 2023 · 3 comments
Open

Deviation from OCPP specification #37

agruenb opened this issue Feb 25, 2023 · 3 comments
Labels
documentation Improvements or additions to documentation

Comments

@agruenb
Copy link

agruenb commented Feb 25, 2023

First of all, thank you for the cool project. However I think I found a slight deviation from the OCPP 1.6 specification. Under 4.1.1. Synchronicity it says:

A Charge Point or Central System SHOULD NOT send a CALL message to the other party unless all the CALL messages it sent before have been responded to or have timed out.

So setting the callConcurrency on client or server to anything different from 1 would violate the spec.

I could see someone not as familiar with the spec increase this option to improve performance in a test environment. However in the wild a microcontroller might run into unforseen problems when receiving multiple messages.

But maybe I understood this wrong which could also be the case 😛

@mikuso
Copy link
Owner

mikuso commented Feb 25, 2023

Hi, thanks for the kind words. 😊

You are correct, a call concurrency above 1 is not allowed in the current versions OCPP, but I specifically wanted to allow the option to support concurrency so that this module could be more useful for other applications. Personally, I use this module for other IoT projects which are not related to EV charging, and this is very useful for that.

I think the best thing for me to do would be to add a note/warning in the readme advising against using a number above 1, as you are correct this can cause compatibility issues, race conditions, etc...

@mikuso mikuso added the documentation Improvements or additions to documentation label Feb 25, 2023
@ZOUHIR23
Copy link

Hello dear @mikuso

please I have a question about your solution when I do the test with a real charging station it can not connect via websocket communication I do not know is what because of localhost I use or something else if you want to help me
thank you in advance

@mikuso
Copy link
Owner

mikuso commented Apr 25, 2023

@ZOUHIR23 this is not related to this issue. Please do not continue to steer issues off-topic.

I would like to help you, but this is not the correct way to ask for help. Please open a new issue instead.

When you open your issue, please include more detail such as what make & model of charging station you are using, and details of how you have configured it (screenshots if possible). This would be a good starting point.

Thanks for your cooperation.

Repository owner locked as off-topic and limited conversation to collaborators Apr 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

3 participants