Skip to content

Conversation

vmx
Copy link
Contributor

@vmx vmx commented Apr 20, 2025

Related Issues

Proposed Changes

So the fix is to simply update to ledger-filecoin-go v1.0.1, which is all that this PR does.

Additional Info

When using some Ledger Nano hardware wallets, some commands fail with:

ERROR: finding ledger: LedgerHID device (idx 0) not found.
Ledger LOCKED OR Other Program/Web Browser may have control of device.

Examples are lotus wallet new secp256k1-ledger and lotus-shed ledger list.

This issue was fixed upstream in Zondax/ledger-go#40, which then got pulled in Zondax/ledger-filecoin-go#38, which was released as https://github.com/Zondax/ledger-filecoin-go/releases/tag/v1.0.1.

Checklist

Before you mark the PR ready for review, please make sure that:

When using some Ledger Nano hardware wallets, some commands fail with:

    ERROR: finding ledger: LedgerHID device (idx 0) not found.
    Ledger LOCKED OR Other Program/Web Browser may have control of device.

Examples are `lotus wallet new secp256k1-ledger` and `lotus-shed ledger list`.

This issue was fixed upstream in Zondax/ledger-go#40,
which then got pulled in Zondax/ledger-filecoin-go#38,
which was released as https://github.com/Zondax/ledger-filecoin-go/releases/tag/v1.0.1.

So the fix is to simply update to `ledger-filecoin-go` v1.0.1, which is all
that this PR does.
@rjan90 rjan90 moved this from 📌 Triage to 🔎 Awaiting Review in FilOz Apr 21, 2025
Copy link
Contributor

@rjan90 rjan90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much for the PR 🙏

@github-project-automation github-project-automation bot moved this from 🔎 Awaiting Review to ✔️ Approved by reviewer in FilOz Apr 21, 2025
@rjan90 rjan90 merged commit 47a24de into filecoin-project:master Apr 21, 2025
94 checks passed
@github-project-automation github-project-automation bot moved this from ✔️ Approved by reviewer to 🎉 Done in FilOz Apr 21, 2025
@vmx vmx deleted the fix-ledger branch April 21, 2025 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🎉 Done
Development

Successfully merging this pull request may close these issues.

2 participants