openhab / org.openhab.binding.zwave

openHAB binding for Z-Wave
Eclipse Public License 2.0
171 stars 202 forks source link

Secure inclusion fails at 'step SECURITY_NETWORK_KEY_SET' #234

Closed brstein closed 7 years ago

brstein commented 7 years ago

Testsetup: OpenHAB2, daily build # 567 (dated Nov., 4th) with Aeotec Z-Stick Gen5 and AeonLabs Multisensor 6

Habmin was set to do a secure inclusion on any device. The sensor was clicked twice on the include button to do a secure include. The inclusion process was logged in the following file, where NODE 7 is the node to be included:

openhab.log.txt

The inclusion process fails (at the bottom of the logfile) with

2016-11-15 22:04:40.577 [ERROR] [curityCommandClassWithInitialization] - NODE 7: SECURITY_INCLUSION_FAILED Failed at step SECURITY_NETWORK_KEY_SET: 10000ms passed since last request was sent, secure inclusion failed.

Before this final failure occurs, the following is stated in the log:

2016-11-15 22:04:29.896 [ERROR] [ssage.ApplicationCommandMessageClass] - NODE 7: Unknown command class 0xffffff98

I am not sure if I have clumsy fingers or if this is a bug. Any help is appreciated,

brstein

cdjackson commented 7 years ago

Security is not currently supported in the OH2 binding. It's being worked on at the moment, but the current binding doesn't have support.

I will close this for now - once the security classes are added (very soon I hope now) we can look at this again if things don't work, but there is no point in keeping this open as the security implementation is a total rewrite from the OH1 implementation.

brstein commented 7 years ago

Thank you for this information. I'm currently very curious about the security features - Please announce it, if the new security classes are ready for testing. Regards, brstein

cdjackson commented 7 years ago

Will do. It's not available yet as I'm working on it at the moment but the initial key exchange is working so I hope it will be soon.

Sent from my iPhone

On 23 Nov 2016, at 12:17, brstein notifications@github.com wrote:

Thank you for this information. I'm currently very curious about the security features - Please announce it, if the new security classes are ready for testing. Regards, brstein

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or mute the thread.