rfjakob / gocryptfs

Encrypted overlay filesystem written in Go

Home Page:https://nuetzlich.net/gocryptfs/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Trezor does not work

pontostroy opened this issue · comments

Fedora 28

lsusb
Bus 002 Device 003: ID 413c:2513 Dell Computer Corp. internal USB Hub of E-Port Replicator
Bus 002 Device 002: ID 8087:0024 Intel Corp. Integrated Rate Matching Hub
Bus 002 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub
Bus 001 Device 005: ID 534c:0001 SatoshiLabs Bitcoin Wallet [TREZOR]
gocryptfs -init -trezor cipher
Choose a password for protecting your files.
Trezor device is not found. Check the connection.
 gocryptfs -version
gocryptfs v1.6-4-g22906cc enable_trezor; go-fuse v20170619-48-g1d35017; 2018-09-03 go1.11

https://wallet.trezor.io and u2f with trezor works fine.

Hmm, looks ok here:

$ gocryptfs -version
gocryptfs v1.6-8-gca24c20 enable_trezor; go-fuse v20170619-48-g1d35017; 2018-09-08 go1.10.3

$ gocryptfs -init -trezor a
Choose a password for protecting your files.

Your master key is:

    11a69a45-9fc661d3-31bfe385-22017ae3-
    389b128d-8a8268d7-e9c29d09-861be33e

If the gocryptfs.conf file becomes corrupted or you ever forget your password,
there is only one hope for recovery: The master key. Print it to a piece of
paper and store it in a drawer. This message is only printed once.

The gocryptfs filesystem has been created successfully.
You can now mount it using: gocryptfs a MOUNTPOINT

What version of ~/go/src/github.com/xaionaro-go/cryptoWallet do you have? I have this

$ git -C ~/go/src/github.com/xaionaro-go/cryptoWallet show
commit 2b98d56e0f5655bd5ddb918b82f3c2f4e528b3b9 (HEAD -> master, origin/master, origin/HEAD)
Author: Dmitry Yu Okunev <dyokunev@ut.mephi.ru>
Date:   Thu Jul 5 23:45:12 2018 +0300

    Export ErrNotSupportedPlatform
git -C ~/GO/GOPATH/src/github.com/xaionaro-go/cryptoWallet show
commit 47f9f6877e4324a8bc47fc5661c32d2fe6d29586 (HEAD -> master, origin/master, origin/HEAD)
Author: Dmitry Yu Okunev <dyokunev@ut.mephi.ru>
Date:   Sat Jul 14 15:18:33 2018 +0300

    Added "Trezor T" to the README.md

diff --git a/README.md b/README.md
index 2150bef..aeb0814 100644
--- a/README.md
+++ b/README.md
@@ -7,6 +7,7 @@ Supported devices:
 | Vendor | Product | Product (lsusb) | Vendor ID | Product ID | Version | Notes |
 | ------ | ------- | --------------- | --------- | ---------- | ------- | ----- |
 | SatoshiLabs | Trezor One | Bitcoin Wallet [TREZOR] | 0x534c | 0x0001 | 1.6.0 |Tested on Linux |
+| InterBiometrics | Trezor T | TREZOR | 0x1209 | 0x53c1 | | Tested on Linux |
 
 ```go
 [...]

git checkout 2b98d56e0f5655bd5ddb918b82f3c2f4e528b3b9 does not help

I was wrong, looks like
git checkout 33e9727d9d249d72970e3558c3ae45fb9b0f1070
Previous HEAD position was 47f9f68 Added "Trezor T" to the README.md
HEAD is now at 33e9727 Started to implement support of "Trezor T

breaks trezor one support

Sorry for the delay. I'll try to look at it on this week.

@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.