SeedSigner / seedsigner

Use an air-gapped Raspberry Pi Zero to sign for Bitcoin transactions! (and do other cool stuff)
MIT License
727 stars 170 forks source link

[Feature Request] Enclosure for WaveShare 19739 Battery HAT Extension #628

Closed kokokale closed 1 week ago

kokokale commented 1 week ago

I'm not sure if there has been any previous discussion around adding on-board power to the SeedSigner or not, but I would personally really love the ability to go entirely wireless with this device. Hardware and 3D modeling are not one of my skillsets, but I believe that with some minimal modifications we can extend existing SeedSigner enclosures to fit this WaveShare Power-Supply HAT: https://www.waveshare.com/ups-hat-c.htm (See images in product listing for an example of this HAT stacked with the 1.3" WaveShare display SeedSigner currently uses)

I can see a few things to consider here:

Since this design is currently untested, I understand if we would like proof-of-concept before proceeding. This request is mainly a request for comment and to gauge interest. I am currently in the process of setting up my SeedSigner, and after everything is functional, I would be happy to purchase and test this hardware for myself and report back my results.

Rob0xFF commented 1 week ago

Please see my enclosue here for reference, hope this might help. Battery monitoring would be great.

Edit: withdrew contribution due to possible Security issues it might impose, See below

SeedSigner commented 1 week ago

I appreciate your interest and creativity here, but the project will likely never officially support any kind of build that includes a battery. The assurances around a SeedSigner's stateless nature arise out of power being completely disconnected from the device, so that the data in memory can clear. Adding a battery also adds the potential for a trickle of power that could cause the device to retain information about private keys that have been created or used with the device.

kokokale commented 1 week ago

Thank you for the response. I understand the security concerns with this model, so I won't create any MRs to this project for battery support.