-
Notifications
You must be signed in to change notification settings - Fork 29
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
nk3 fido2 status #453
Comments
Historically, the I’m not sure what’s the best way to solve this. Ideally, we would have separate subcommands for the FIDO2 protocol and the FIDO2 device, but how would we call them? Maybe As a first improvement, we could add messages to the problematic subcommands indicating that they are only for Nitrokey FIDO2 devices. |
Interesting, I was not aware the FIDO2 protocol does not know a If you think about refactoring nomenclature, consider prepending "nk-" to the respective commands, for example change it to NB I saw this fido2-hmac-secret support question recently, which refers to another example of a custom FIDO2 command. While |
I'd like to make yet another proposition regarding the command nomenclatura. What led to me opening the issue was not being aware of what's FIDO spec standard/NK FIDO specific. If the
Another solution may be that nitropy detects the hardware model and prints something like "This subcommand is not part of the FIDO2 specification and implemented specifically for Nitrokey FIDO2 devices.", but for me as a user the --help info would have sufficed. By the way, "subcommand" in the helptext should generally be renamed "command", as this help only lists "Commands:". |
Hi,
new nk3 A with factory fw1.5.0 and 41-udev rules. 'nitropy nk3 test' and
nitropy fido2 verify
work as expected.Executing
nitropy fido2 status
results inCritical error: An unhandled exception occurred Exception encountered: CtapError('CTAP error: 0x01 - INVALID_COMMAND')
It also fails after doing a fido2 reset, with/without PIN, I tried stopping pcscd too.
nitropy.log.r8w6lsar.txt
The text was updated successfully, but these errors were encountered: