Support for Ledger hardware wallets

[pipermerriam]

What was wrong?

Hardware wallets are generally considered the gold standard for private key security. Supporting them in web3.py would allow users to take advantage of that security.

How can it be fixed?

Figure out how to do signing with a ledger hardware wallet.

https://github.com/LedgerHQ/blue-loader-python

This repository from the Ledger github is probably a good starting point.

Definition of Done

• Something which conforms to the LocalAccount API used by web3.eth.account.
• An easy way to detect a connected to a ledger wallet.
• Should work for Nano-S
• Support for Blue not required but encouraged.
• Integration test suite which can be run with a connected ledger device.

[voith]

Integration test suite which can be run with a connected ledger device.

Curious to Know how the device would be configured with CI to run integration tests.

[pipermerriam]

These tests likely wouldn't be run in CI. They'd be something we'd run manually in a local environment.

[voith]

registering this issue for myself. I've spoken to @miohtama over chat about this!

[pipermerriam]

@voith assuming that you have a ledger to test against?

[voith]

May be a little early to say but most likely I will get sponsored to get this feature out. To answer your question I don't own one at the moment, but if everything goes as planned I should have one by the end of this week.

[carver]

Any code for this issue should probably end up in https://github.com/ethereum/eth-account

[petri]

Something that could perhaps be worth taking a look at: https://github.com/bargst/pyethoff

[bargst]

I extracted relevant part of my repo into https://gist.github.com/bargst/5f896e4a5984593d43f1b4eb66f79d68
I could try to turn it into eth-account code but I may need guidance !

[pipermerriam]

Made an issue for this in eth-account: ethereum/eth-account#25

[pipermerriam]

@bargst you appear to be on the right track. I'd take a look at the linked issue for hardware wallets: ethereum/eth-account#25 and what may potentially be a dependency on that ethereum/eth-account#24