-
Notifications
You must be signed in to change notification settings - Fork 150
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
Does KeepKey support GPG? #440
Comments
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? |
FTR, Keepkey support was added in 5932a89 when it didn't support GPG. |
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. |
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. |
CC: @keepkey @pastaghost @markrypto @markrypt0 @mrnerdhair @bgok (KeepKey maintainers) |
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
orgpg_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.
The text was updated successfully, but these errors were encountered: