SeedSigner / seedsigner

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

[Enhancement] Update seed_phrase_to_qr.py; warnings, add Compact SeedQR #523

Closed kdmukai closed 9 months ago

kdmukai commented 9 months ago

Description

The tools/seed_phrase_to_qr.py utility script converts a mnemonic to a digital SeedQR in png format. The tool is only meant for dev testing purposes and should announce itself as such if someone tries to run it.

Also adds support for generating Compact SeedQR format.

Changes mnemonic input to use interactive input steps rather than including the mnemonic when executing from the command line (minor improvement to not have mnemonics in your bash history, though, again, only disposable test seeds should be used here).

This pull request is categorized as a:

Checklist

If you modified or added functionality/workflow, did you add new unit tests?

I have tested this PR on the following platforms/os:

Meant to run in local dev environment, not on Raspi.

newtonick commented 9 months ago

LGTM, tested without issue