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

Support non-smartcard (HID feature) interface, including OnlyKey #2

Open
onlykey opened this issue Feb 20, 2019 · 2 comments
Open

Support non-smartcard (HID feature) interface, including OnlyKey #2

onlykey opened this issue Feb 20, 2019 · 2 comments
Assignees

Comments

@onlykey
Copy link

onlykey commented Feb 20, 2019

Following up from another discussion. I would like to discuss further supporting OnlyKey. Probably the first thing to mention is OnlyKey currently does not have a CCID usb interface. It supports challenge-response the same way as Yubikey which actually uses a USB keyboard interface. The communication is accomplished by sending HID feature reports (8 bytes of data at a time). One advantage of this approach is no UDEV rule is required as USB HID keyboards are supported natively.

@crosser crosser changed the title OnlyKey Support Support non-smartcard (HID feature) interface, including OnlyKey Feb 20, 2019
@crosser
Copy link
Owner

crosser commented Feb 20, 2019

Thank you for the suggestion.
I generally disapprove of using USB-HID interface in a security gadget, because that

  1. makes it unusable in other-than-desktop context (tablets and phones), and
  2. potentially increases attack surface.

That said, supporting a pupular interface that is also used by low-end yubikeys would be a useful improvement to this PAM module, so I am keeping this open as a potential TODO item.
Thanks again!

@crosser crosser self-assigned this Feb 20, 2019
@onlykey
Copy link
Author

onlykey commented Feb 20, 2019

@crosser Another common interface used by security devices is USB HID (FIDO U2F). This USB HID interface is different than using the keyboard interface. It generally communicates via 64bytes in/out. This would also be something to consider supporting as this is supported by many devices. Yubico has a PAM-U2F module.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants