-
Notifications
You must be signed in to change notification settings - Fork 262
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
Trezor does not work #261
Comments
Cc @xaionaro |
Hmm, looks ok here:
What version of ~/go/src/github.com/xaionaro-go/cryptoWallet do you have? I have this
|
git checkout 2b98d56e0f5655bd5ddb918b82f3c2f4e528b3b9 does not help |
I was wrong, looks like breaks trezor one support |
Sorry for the delay. I'll try to look at it on this week. |
Trezor support has been broken since Sept 2018 ( #261 ). Disable trezor.go by renaming to trezor.go.broken. This keeps "dep" from having to pull in A LOT OF dependencies: Before: $ du -sh vendor/ 49M vendor/ After: $ du -sh vendor/ 16M vendor/
@rfjakob I suggest we close this and remove Trezor support completely from the master. Recently, we added FIDO2 into Trezor and it makes sense to use the FIDO2 HMAC Secret Extension instead. This has the advantage that all FIDO2 tokens will be supported in one unified way. |
Damn, forgot about this ticket :( So it's not actual anymore? |
Sounds good. Let's drop this and move to fido2 hmac. |
Fedora 28
https://wallet.trezor.io and u2f with trezor works fine.
The text was updated successfully, but these errors were encountered: