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.

Sign up for GitHub

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

Jump to bottom

Allow arbitrary methods in expert mode #65

Closed
Stebalien opened this issue Jan 5, 2021 · 4 comments · Fixed by #74
Closed

Allow arbitrary methods in expert mode #65

Stebalien opened this issue Jan 5, 2021 · 4 comments · Fixed by #74
Assignees
@jleni
Labels
P2-Mid Medium Priority

Comments

@Stebalien
Copy link

Currently, only methods 0-7 are allowed. Unfortunately, this means:

  1. Ledgers cannot be used for miner "owner" keys.
  2. Some multisig methods may not be allowed either.
This was referenced Jan 5, 2021
Closed
Closed
@jleni jleni added the P2-Mid Medium Priority label Feb 20, 2021
@cwhiggins
Copy link

Is there an ETA on Ledger support for "owner" keys & multisig?

@hyunmoon
Copy link

Very curious as well.

@jennijuju jennijuju mentioned this issue Apr 14, 2021
Closed
@mikereinhart
Copy link

We're also interested in seeing this issue resolved, particularly the multisig methods - thanks.

@jleni jleni self-assigned this Jun 5, 2021
@ohJohanZ
Copy link

ohJohanZ commented Jun 7, 2021 •
edited
Loading

when I run:
lotus-miner actor set-owner --really-do-it f1xxxxxxxxxxxxx <ledger key>
It will return error, "ERROR: mpool push: failed to sign message: Unexpected method"
Please fix this issue. @jleni

@jleni jleni linked a pull request Jun 9, 2021 that will close this issue
Add support for variable methods #74
Merged
@jleni jleni closed this as completed Jun 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jleni jleni

Labels
P2-Mid Medium Priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add support for variable methods Zondax/ledger-filecoin
6 participants
@Stebalien @hyunmoon @mikereinhart @ohJohanZ @jleni @cwhiggins