Does KeepKey support GPG?
doolio opened this issue · 10 comments
trezor-agent/agents/keepkey/setup.py
Line 7 in 3c911e9
Suggests it does.
trezor-agent/agents/keepkey/setup.py
Lines 35 to 37 in 3c911e9
Yet, there appears to be no GPG entry points defined.
trezor-agent/agents/keepkey/keepkey_agent.py
Lines 1 to 5 in 3c911e9
L1 imports the GPG module but there are then no gpg_tool
or gpg_agent
lambda function assignments like we have for the other hardware devices agents.
trezor-agent/libagent/device/keepkey.py
Line 10 in 3c911e9
Suggests only SSH is supported.
If you have one, you can simply enable the agent and test.
I only have a Trezor T.
@keepkeyjon IIUC, GPG wasn't supported initially on Keepkey - and was added only in keepkey/keepkey-firmware@95e41aa (#8), right?
That would explain the current state. Is it a matter of simply adding the entry points and the lambda functions I discuss above? If so I can submit such a PR for consideration.
Is it a matter of simply adding the entry points and the lambda functions I discuss above?
Not sure - it should also be tested with the latest Keepkey firmware, to make sure it indeed works.
Hmm ok. I only have a trezor T so can't help with such tests.
The best motivation for contribution is needing it, or knowing someone who needs it. Worrying about support for a device you don't have and don't know anyone who has is noble, but most often not particularly productive.
You're right of course. I'm just highlighting potential issues where I see them.
@keepkeyjon IIUC, GPG wasn't supported initially on Keepkey - and was added only in keepkey/keepkey-firmware@95e41aa (#8), right?
CC: @keepkey @pastaghost @markrypto @markrypt0 @mrnerdhair @bgok (KeepKey maintainers)
Could you please let us know if GPG signature/decryption is supported by KeepKey?