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

New NIP: Nostr BLE Signing Device Protocol #1706

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

cr0bar
Copy link

@cr0bar cr0bar commented Jan 20, 2025

This PR is to introduce support for signing and encryption/decryption on hardware based Nostr Signing Devices over BLE.

@cr0bar
Copy link
Author

cr0bar commented Jan 21, 2025

View NIP

@staab
Copy link
Member

staab commented Jan 21, 2025

This is cool, but I'm not sure what all the key management methods are for. Usually it's the responsibility of the signer to provide an interface for this, although there has been talk of a hook in NIP 07 to notify clients of when the account has changed. I would also try to use the same method names as exist in 07/55/46 when possible.

@cr0bar
Copy link
Author

cr0bar commented Jan 21, 2025

@staab the idea behind this was to allow a client to switch between and manage public keys held on the device, however, I see your point. In reality the only commands the need to be exposed to the client over BLE are:

  • /public-key
  • /sign-message
  • /encrypt-message-nipXX
  • /decrypt-message-nipXX
  • /ping

I'll review and reflect on the other NIPs you mentioned.

@vitorpamplona
Copy link
Collaborator

As a Bluetooth dev for 5 years, I am a big fan of this.

We can have earplugs that read the permission and sign for you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants