polkadot-js / apps

Basic Polkadot/Substrate UI for interacting with a Polkadot and Substrate node. This is the main user-facing application, allowing access to all features available on Substrate chains.
https://dotapps.io
Apache License 2.0
1.75k stars 1.54k forks source link

WebUSB connection to Ledger on Desktop app #4803

Open DanielHirsbrunner opened 3 years ago

DanielHirsbrunner commented 3 years ago

Hello together

I'm using the Desktop app version 0.83.1 on a windows 10 Computer (10.0.19042) and would like to connect my ledger via WebUsb to add my accounts.

When adding an Account I got the message "No device selected"

If I do the same in the Web Version, chrome shows me an PopUp to select the device I would like to connect to the site, I confirm it and then the connection works. I checked this for Kusama and Polkadot accounts and also installed the app on a second computer.

Is it possible that electron needs some further configuration to support the webusb connection to Ledger and it therefore fails?

Thanks for your help!

ConnectLedger

jacogr commented 3 years ago

It does indeed sound like a permission issue - will do some reading to see what is required in this environment.

jleni commented 3 years ago

AFAIK, if you are wrapping things with electron, you will need to use HID transport instead.

DanielHirsbrunner commented 3 years ago

Thanks for your Inputs, is there any plan to adjust the provided Desktop Wallet to support ledger in the near future?

jacogr commented 3 years ago

Will get there. The focus is elsewhere atm.

jacogr commented 3 years ago

This issue has been open for 21 days with no activity and is not labelled as an enhancement. It will be closed in 7 days.

DanielHirsbrunner commented 3 years ago

IMHO this is a Bug and I would use the Desktop Wallet if this Bug wouldn't exist. But if I'm the only one interested then probably you can close the ticket.

jacogr commented 2 years ago

node-hid is enabled in the current build, but doesn't work.

It needs some additional config, see - https://github.com/node-hid/node-hid#electron-projects-using-node-hid