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

Does KeepKey support GPG? #440

Open
doolio opened this issue Sep 11, 2023 · 10 comments
Open

Does KeepKey support GPG? #440

doolio opened this issue Sep 11, 2023 · 10 comments

Comments

@doolio
Copy link
Contributor

doolio commented Sep 11, 2023

description='Using KeepKey as hardware SSH/GPG agent',

Suggests it does.

entry_points={'console_scripts': [
'keepkey-agent = keepkey_agent:ssh_agent',
]},

Yet, there appears to be no GPG entry points defined.

import libagent.gpg
import libagent.ssh
from libagent.device import keepkey
ssh_agent = lambda: libagent.ssh.main(keepkey.KeepKey)

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.

if protocol not in {'ssh'}:

Suggests only SSH is supported.

@SlugFiller
Copy link
Contributor

If you have one, you can simply enable the agent and test.

@doolio
Copy link
Contributor Author

doolio commented Sep 14, 2023

I only have a Trezor T.

@romanz
Copy link
Owner

romanz commented Sep 14, 2023

@keepkeyjon IIUC, GPG wasn't supported initially on Keepkey - and was added only in keepkey/keepkey-firmware@95e41aa (#8), right?

@romanz
Copy link
Owner

romanz commented Sep 14, 2023

FTR, Keepkey support was added in 5932a89 when it didn't support GPG.

@doolio
Copy link
Contributor Author

doolio commented Sep 14, 2023

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.

@romanz
Copy link
Owner

romanz commented Sep 14, 2023

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.

@doolio
Copy link
Contributor Author

doolio commented Sep 14, 2023

Hmm ok. I only have a trezor T so can't help with such tests.

@SlugFiller
Copy link
Contributor

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.

@doolio
Copy link
Contributor Author

doolio commented Sep 15, 2023

You're right of course. I'm just highlighting potential issues where I see them.

@romanz
Copy link
Owner

romanz commented Sep 17, 2023

@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?

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

No branches or pull requests

3 participants