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.

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

Disable U2F #15

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Disable U2F #15

wants to merge 1 commit into from

Conversation

justicz
Copy link
Contributor

@justicz justicz commented Feb 27, 2020

Adding this draft PR here to be merged once we can test that it doesn't break anything on the Nano X

@zeldovich
Copy link
Collaborator

@mxmauro might be sad about this, though this may be the right plan if Ledger thinks the U2F tunneling is a dead end.

@mxmauro
Copy link

mxmauro commented Feb 27, 2020

Hahaha seems the Windows/WebUSB issue is now fixed. We are doing some tests and, at first glance, seems to be related to some fix on firmware 1.6.

Because on this, we are also planning to drop support to U2F on the web wallet but our initial tests indicates Chrome works fine but Firefox and Opera does still not support WebUSB.

@mxmauro
Copy link

mxmauro commented Mar 4, 2020

Hi guys, a feedback:

On Chrome/Windows, confirmed, WebUSB is working now. But Firefox, Safari and Opera Mini still does not support WebUSB and uses U2F. Also no plans to support WebUSB (Mozilla guys says it is harmful)

zeldovich pushed a commit that referenced this pull request Feb 2, 2021
gmalouf pushed a commit that referenced this pull request Sep 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants