-
-
Notifications
You must be signed in to change notification settings - Fork 34
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
security feature not working #9
Comments
Hi @enmanoj, indeed the crypto support has not been ported. Feel free to submit a patch and I will review it, otherwise I hope to get to it soon. |
Hi @enmanoj - could you please post a failure case for this? It will speed up reproduction so I can get a fix done. |
You need to add 'secure' while defining characteristic to enable security. To do that please follow the below steps. Execute the peripheral.py file You will get the below error. This error is due to typo error in file
Change Output
Once you fixed the above run peripheral.py and read the same again you will find this error
Output
Note: Am using nRF Connect for Mobile as master for reading and writing data. |
Hi @enmanoj, I've committed a fix for some syntactical issues that you identified, but the crypto stuff is not yet implemented - I haven't been able to exercise it. Can you reproduce this same issue using either Windows, MacOS or iOS, and post the repro scenario here? I don't have Android available to me. |
Is there any way to bypass this secure feature? I'm running the echo and battery services without any change and the program is stuck at the crypto point. |
Hi @shrsulav - can you switch to the 'transpiled' branch and try that? Those bugs are all resolved on that branch. |
Hello @Adam-Langley, I checked out the transpiled branch and ran the main.py of the echo example. And, this is the error I'm getting:
|
Hi @shrsulav have you followed the instructions under 'Running The Examples'? |
Another option is: This Installs the project in editable mode from the current project repositry. This will link the package to the local files meaning any changes locally would be available in your environment |
@Adam-Langley I cloned the repo, checked out to the transpiled branch and on using the command I got the following error: |
when using 'secure': ['write'] while initialling Characteristics getting a run-time error
File "/usr/local/lib/python2.7/dist-packages/pybleno/hci_socket/Smp.py", line 101, in handlePairingConfirm
self._r = crypto.r()
NameError: global name 'crypto' is not defined
Please fix a patch for this.
The text was updated successfully, but these errors were encountered: