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

Investigate potential Ledger integration #2

Open
snoyberg opened this issue Apr 17, 2024 · 5 comments
Open

Investigate potential Ledger integration #2

snoyberg opened this issue Apr 17, 2024 · 5 comments

Comments

@snoyberg
Copy link
Member

It would be convenient to be able to sign transactions using a Ledger hardware wallet. Currently such signing tends to happen in web apps instead. A very quick investigation pointed me to this:

https://lib.rs/crates/ledger-cli

@nubis
Copy link

nubis commented Apr 17, 2024

I may get into this.
But I thought this cosmos-rs library was for backend services. What would be the use case for signing with ledger? Is it for hooking up a ledger as an unattended HSM? If so, the unattended part is tricky, I doubt the generic cosmos app supports this, so a new app may be needed.

@snoyberg
Copy link
Member Author

This would be for command line usage. The main reason for this is to be able to do things like signing multisig proposals from the CLI without needing to use some web app. And the reason to avoid the web apps is that we've had lots of cases where the web apps have broken due to node issues or chain API breakages, while the CLI tooling tends to be much more reliable.

@nubis
Copy link

nubis commented Apr 17, 2024

oh, I can look at it, what's the priority?

Also: I could probably add trezor too.

@snoyberg
Copy link
Member Author

Low priority, a nice to have when we don't have other things to do.

I don't think Trezor has Cosmos support.

@nubis
Copy link

nubis commented Apr 18, 2024

got it, I'll keep it in the backburner. Regarding trezor (and other hw wallets) and Cosmos, I guess it will depend on which cosmos blockchain we're talking about, most of the wallets can be coherced to at least sign Secp256k1.

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

2 participants