ledger hardware wallet with newer firmware versions aren't supported anymore

[vchong]

tacchaind keys add foo --ledger used to work on older firmware versions but now it just returns

Error: failed to generate ledger key
...
2:49AM ERR failure when running app err="failed to generate ledger key"

ledger
model: nano x
firmware version: 2.5.0, 2.6.0

[tgntr]

hello, thanks for reporting! are you able to test and/or open issue in cosmos-sdk repo please?

here's my suggested steps:

1. test with cosmos-sdk v0.50.13 simapp binary and confirm the error can be reproduced
2. test with cosmos-sdk v0.53 simapp binary and check if error persists or is fixed
3. if error persists there, this should be reported there to get a discussion going

[vchong]

simd works

$ ./build/simd version
0.50.13

$ ./build/simd keys add foo --ledger
...

- address: cosmos1...
  name: foo
  pubkey: '{"@type":"/cosmos.crypto.secp256k1.PubKey","key":"A..."}'
  type: ledger

so something off in https://github.com/TacBuild/tacchain/blob/v1.0.2/go.mod#L273-L274?

	// replace to cosmos-sdk fork for liquid stake support. See: https://github.com/TacBuild/cosmos-sdk/pull/2
	github.com/cosmos/cosmos-sdk => github.com/TacBuild/cosmos-sdk v0.0.0-20251010131633-e43d8d10a9a9

[tgntr]

thanks for checking! could you please test with our previous version (v1.0.1 - https://github.com/TacBuild/tacchain/releases/tag/v1.0.1) to see if it's caused by cosmos-sdk replace which happened with latest v1.0.2 version

ledger devices used to work with our binary for sure, but i don't have a ledger device at hand to test myself

[vchong]

you're welcome! so v1.0.1 doesn't work (same error as v1.0.2), which is weird, because it has the same cosmos-sdk replace as v0.0.9, which works

update: because the ledger firmware is too new

[vchong]

you're welcome! so v1.0.1 doesn't work (same error as v1.0.2), which is weird, because it has the same cosmos-sdk replace as v0.0.9, which works

so i just tried tacchaind keys add foo --ledger with v0.0.9 and it didn't work either! 🤔

update: because the ledger firmware is too new

[vchong]

I tried keys add --ledger again with version 1.0.1 and an older ledger, and it worked! So this brings us back to issue originally reported, i.e. the tacchaind binary doesn't work with ledgers with newer firmware versions!

[vchong]

@tgntr this was fixed with:

diff --git a/wallets/usbwallet/hub.go b/wallets/usbwallet/hub.go
index 897ab9f..ea1de1e 100644
--- a/wallets/usbwallet/hub.go
+++ b/wallets/usbwallet/hub.go
@@ -64,6 +64,15 @@ func NewLedgerHub() (*Hub, error) {
 		0x0005, /* Ledger Nano S Plus */
 		0x0006, /* Ledger Nano FTS */
 
+		0x0007, /* Ledger Flex */
+
+		0x0000, /* WebUSB Ledger Blue */
+		0x1000, /* WebUSB Ledger Nano S */
+		0x4000, /* WebUSB Ledger Nano X */
+		0x5000, /* WebUSB Ledger Nano S Plus */
+		0x6000, /* WebUSB Ledger Nano FTS */
+		0x7000, /* WebUSB Ledger Flex */
+
 		0x0015, /* HID + U2F + WebUSB Ledger Blue */
 		0x1015, /* HID + U2F + WebUSB Ledger Nano S */
 		0x4015, /* HID + U2F + WebUSB Ledger Nano X */

in https://github.com/TacBuild/evm/blob/124c5baba48e/ on https://github.com/TacBuild/tacchain v1.0.1.

This patch derived from https://github.com/cosmos/evm/pull/690/changes#diff-71219c6822946c781ca2fe51e99cc830206038528ce7b3af7c4797e735aead6e.

As mentioned by one of your folks, the full fix would be to bump deps in go.mod:

• zondax/ledger-go -> v1.0.1
• cosmos/ledger-cosmos-go -> v0.16.0

and cherry-pick code fixes from below into https://github.com/TacBuild/evm:

• fix: ledger nano x on linux cosmos/evm#862 - Ledger Nano X device matching
• fix: allow ledger flag work with coin type 60 cosmos/evm#690 - Coin type 60 support

Do you plan to incorporate these into the next (v1.0.4) release?

[AkKoks]

@tgntr this was fixed with:

diff --git a/wallets/usbwallet/hub.go b/wallets/usbwallet/hub.go
index 897ab9f..ea1de1e 100644
--- a/wallets/usbwallet/hub.go
+++ b/wallets/usbwallet/hub.go
@@ -64,6 +64,15 @@ func NewLedgerHub() (*Hub, error) {
 		0x0005, /* Ledger Nano S Plus */
 		0x0006, /* Ledger Nano FTS */
 
+		0x0007, /* Ledger Flex */
+
+		0x0000, /* WebUSB Ledger Blue */
+		0x1000, /* WebUSB Ledger Nano S */
+		0x4000, /* WebUSB Ledger Nano X */
+		0x5000, /* WebUSB Ledger Nano S Plus */
+		0x6000, /* WebUSB Ledger Nano FTS */
+		0x7000, /* WebUSB Ledger Flex */
+
 		0x0015, /* HID + U2F + WebUSB Ledger Blue */
 		0x1015, /* HID + U2F + WebUSB Ledger Nano S */
 		0x4015, /* HID + U2F + WebUSB Ledger Nano X */

in https://github.com/TacBuild/evm/blob/124c5baba48e/ on https://github.com/TacBuild/tacchain v1.0.1.

This patch derived from https://github.com/cosmos/evm/pull/690/changes#diff-71219c6822946c781ca2fe51e99cc830206038528ce7b3af7c4797e735aead6e.

As mentioned by one of your folks, the full fix would be to bump deps in go.mod:

• zondax/ledger-go -> v1.0.1
• cosmos/ledger-cosmos-go -> v0.16.0

and cherry-pick code fixes from below into https://github.com/TacBuild/evm:

• fix: ledger nano x on linux cosmos/evm#862 - Ledger Nano X device matching
• fix: allow ledger flag work with coin type 60 cosmos/evm#690 - Coin type 60 support

Do you plan to incorporate these into the next (v1.0.4) release?

Greetings, @vchong!

Good news - these fixes are already included in the upcoming major update.

Thanks for the detailed report!