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

Ledger U2F timeout issue #224

Open
TobiaszCudnik opened this issue Apr 15, 2019 · 4 comments · Fixed by #227
Open

Ledger U2F timeout issue #224

TobiaszCudnik opened this issue Apr 15, 2019 · 4 comments · Fixed by #227
Assignees
@TobiaszCudnik

Comments

@TobiaszCudnik
Copy link
Contributor

Current version of ledger has a random timeout issue. At the moment we instruct the user to redo the tx confirmation in case that happens. Also the UI timeout has been lowered from 30sec to 10.

Official info:
https://support.ledger.com/hc/en-us/articles/360018810413-U2F-timeout-in-Chrome-browser
@TobiaszCudnik TobiaszCudnik mentioned this issue Apr 15, 2019
Closed
4 tasks
TobiaszCudnik added a commit that referenced this issue Apr 16, 2019
@TobiaszCudnik
ledger webusb transport test implementation #224
4fe7e7c
@TobiaszCudnik
Copy link
Contributor Author

There's some initial bits for the webusb transport in feature/224-ledger-webusb, but it requires the Rise app on the ledger to explicitly support webusb, as stated in this ticket.

@kjarada
Copy link

kjarada commented Apr 24, 2019

I'm having the same issue, despite the fact that ledger support advised using brave and opera browser but it's still not working in all these platforms with same error U2F Timeout.

@TobiaszCudnik TobiaszCudnik mentioned this issue Apr 25, 2019
Merged
TobiaszCudnik added a commit that referenced this issue Apr 30, 2019
@TobiaszCudnik
webusb ledger #224
22fee00 
- fixed deep linking (button to activate webusb)
- remember last used ledger
- button to unmount ledger
@TobiaszCudnik
Copy link
Contributor Author

We have a fully working webusb version (tested on Chrome, but blink-based browsers should work too) in this PR.

Unfortunately to make it work one needs the new rise app installed on the device and that one still hasn't been released to the ledger app store. Stay tuned...

@TobiaszCudnik TobiaszCudnik mentioned this issue May 1, 2019
Open
@TobiaszCudnik
Copy link
Contributor Author

We're still waiting for the new rise ledger app to be published on ledger manager.

Version working with webusb: 1.3.0
Current version: 1.2.1

Related commit

@TobiaszCudnik TobiaszCudnik self-assigned this Jun 13, 2019
@TobiaszCudnik TobiaszCudnik reopened this Jun 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TobiaszCudnik TobiaszCudnik

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

webusb transport for Ledger Nano RiseVision/rise-react-wallet
2 participants
@TobiaszCudnik @kjarada