Sensebender Gateway with NRF24 PA and Hardsigning and Encryption
-
@Anticimex I'm sorry, my vacation brain just cannot comprehend all these terms (I don't think it would even if I wasn't on vecaction :) My four hour experience with c/c++ surely has its limits).
Is it safe to say, the GW sketch does not use the ATSHA chip, and the sketch needs some modification for it to start using it?
In the mean-time, I'll try it out with the default sketch. It looks like the GW and the bender are talking to each other
@Magnus-Pernemark no, I am saying that the sketch use the atsha by default for self testing. It is not using it by default for signing. And the problem is that if you want to use it for signing it has already included some code for self testing that it shouldn't in that case. If you are not able to code c i suggest hanging on some time for someone to fix the problem. I can't do it at the moment since I am on the road. Ping @tbowmo
-
@Magnus-Pernemark
You need to enable signing with
#define MY_SIGNING_ATSHA204This will make the sketch break though, as it includes the atsha204 driver already (and the library wants to include it as well)
To remove the error comment out the following line (it's line 86 in the original source)
#include <drivers/ATSHA204/ATSHA204.cpp>(Reference to the line in github :
https://github.com/mysensors/MySensors/blob/master/examples/SensebenderGatewaySerial/SensebenderGatewaySerial.ino#L86)As an alternative, you can load the standard GatewaySerial sketch.. The only difference to the SensebenderGatewaySerial, is that the later includes test routines used in the factory.
@Anticimex I don't think that I made a PR for conditionally include the atsha driver, as the error still persists in dev branch..
-
@Magnus-Pernemark
You need to enable signing with
#define MY_SIGNING_ATSHA204This will make the sketch break though, as it includes the atsha204 driver already (and the library wants to include it as well)
To remove the error comment out the following line (it's line 86 in the original source)
#include <drivers/ATSHA204/ATSHA204.cpp>(Reference to the line in github :
https://github.com/mysensors/MySensors/blob/master/examples/SensebenderGatewaySerial/SensebenderGatewaySerial.ino#L86)As an alternative, you can load the standard GatewaySerial sketch.. The only difference to the SensebenderGatewaySerial, is that the later includes test routines used in the factory.
@Anticimex I don't think that I made a PR for conditionally include the atsha driver, as the error still persists in dev branch..
@tbowmo no, but I remember that another user had the same issue and agreed to file a pr but perhaps he never got around to it. Should just be a matter of adding a ifndef though.
Do you feel secure today? No? Start requiring some signatures and feel better tomorrow ;)
-
@tbowmo no, but I remember that another user had the same issue and agreed to file a pr but perhaps he never got around to it. Should just be a matter of adding a ifndef though.
