-
Notifications
You must be signed in to change notification settings - Fork 180
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
OCPP 1.6-J Security #90
Comments
Hi Jim, Sounds exciting. I did noticed that OCA wrote IT om their Twitter profile, but haven't had a chance to look for my self. Does anyone know what has changed? Thomas |
I've had a chance to skim the new "white paper" now:
Jim |
Here is a link to the document and specification. It talks mainly about Basic Authentication and Client Side Certificates: https://www.openchargealliance.org/news/enhanced-security-for-ocpp-16/ |
We could think of splitting this issue into 2 as they could be implemented separately and I guess Basic Authentication is much easier to implement.
|
Hi Thomas, Is there any news on this topic? Next to the authentication methods the document also describes new messages. |
Hi @erikvda Nothing new from me. I haven't gotten around to look into it, and I don't see that I will get around to it in the near future. Sorry. |
@TVolden First of all thanks to you and contributors for such a easy to use library. As @dersteve mentioned the steps. I am pretty interested in having Basic Authentication supported on OCPP json server. Can you please let me know which architectural adjustements we need to make inorder to have this feature. I can work on this but needed some help in terms of architecture wise. Cheers! |
Hi @vikram919, thanks for your reply. I'll look into it at first chance, which may be Tuesday at best. Hope that's okay. |
@TVolden tuesday should be fine. |
Hi again @vikram919, I had some time to look into it today. I'm having a hard time finding a way to use Basic Authentication with the Websocket component I use (Java-websocket). I'm not sure how to make it work. Sorry. Here is a link, that discusses it: |
Hi @TVolden, sorry for being late to the game, but I just created a pull request which adds OCPP-J client-side HTTP basic authentication support to your library. We successfully tested this in a plugfest against backends expecting basic authentication. |
Thanks @robert-s-ubi, I have merged it in now. @vikram919 if your still interested. Can you test if it works now? - and if so, close this issue. Thanks every one. |
@TVolden , thanks for the update, I will test it sooner until by next week and close this issue. |
Hi! First of all, great project and thank you for the effort. Regarding basic authentication on server side, not sure I understand what is the complication here. There is |
Added pull request #163 , please review |
Is there a plan to update the maven repo with this change in near future? |
Hi @dagarwal82 Thanks for asking. I can give it a try after exams. So after the 15th or so. Sincerely, |
I think authenticationSession in ServerEvents should also throw AuthenticationException so that ListenerEvents can capture and rethrow: listener.open(
Line 88 can not throw AuthenticationException and do not validate the outcome either. |
Hi @TVolden, @dagarwal82, I am actually in the process of making an implementation right now. Expect a PR soon... |
See my PR #196. Please take your time to review, it's a big one :-) The library already supports HTTP basic auth and SSLContext is used so security profiles 2 and 3 are in theory already possible. However none of the defined management messages are implemented, which is the scope of my PR. The whitepaper mentioned by @V2G-UK is already outdated in the meantime as a 2nd edition has been released. The whitepaper does not define a "profile" and "features" as the original whitepaper did. However I did create a profile and features similar to the existing ones to fit everything in and be compatible with the rest of the library. |
Hi members, How i can setup the basic auth security profile? Right now I am getting the password values as null. You little help will be much appreaciate. |
The OCPP 1.6 standard has just changed. See:
http://www.v2g-evse.com/2018/12/18/open-charge-alliance-enhances-ocpp-1-6-security/
Jim
The text was updated successfully, but these errors were encountered: