Solana: Trezor wallet is not supported.

Created on 2 Jul 2019  路  11Comments  路  Source: solana-labs/solana

Problem

Trezor wallet is not supported.

Proposed Solution

Add support for the cli to use the wallet.

good first issue stale

All 11 comments

To continue the analogy, can it somehow be built on top of ledger-tool? No, I confused it with Ledger support.

Yeah I'd say either a stand-alone tool to begin with or potentially a subcommand of the cli/

We should keep in mind that more recently Parity have completely removed hardware wallet support. The point being that supporting it was not possible in a non-interactive environment. It looks like the required interactivity had been removed prior to that in a process of decoupling account management from the blockchain engine. It's probably better to try keeping anything that requires interaction inside cli or maybe introduce another subcrate? hw?

@vkomenda can v0 simply proxy over from then cli to the HSM?

@aeyakovenko what is HSM?

Do we need to add Solana support to Trezor firmware?

@vkomenda HSM is just an acronym - Hardware Security Module. For v0, I think we can just use Trezor to store the keys and provide signatures for message that the CLI requests.

Is it OK to limit Trezor support to firmware version 1.7.0 and above? Reference Parity code might have worked up to versions 1.6.x but it's definitely not compatible with versions starting 1.7.0 because USB IDs and some magic numbers changed. I think we might expect users to have upgraded their device firmware to the latest version.

@vkomenda that works for us.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

This stale issue has been automatically closed. Thank you for your contributions.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

garious picture garious  路  10Comments

xtrapower picture xtrapower  路  9Comments

mvines picture mvines  路  10Comments

mvines picture mvines  路  8Comments

garious picture garious  路  12Comments