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

Add support for Bluetooth HID devices to hid_linux #169

Merged
merged 1 commit into from
May 18, 2020

Conversation

fmeum
Copy link
Contributor

@fmeum fmeum commented May 6, 2020

Previously, hid_linux.c would fail to extract product and manufacturer
for Bluetooth HID devices since they are not available as individual
sysattrs.

With this commit, the product name is parsed from the uevent sysattr
while the manufacturer, which is not exposed by the kernel, is set to a
descriptive constant ("Bluetooth HID").

I have verified that with this and #168, libfido2 can speak to and detect WearAuthn:

Screenshots

Previously, hid_linux.c would fail to extract product and manufacturer
for Bluetooth HID devices since they are not available as individual
sysattrs.

With this commit, the product name is parsed from the uevent sysattr
while the manufacturer, which is not exposed by the kernel, is set to a
descriptive constant ("Bluetooth HID").
@martelletto
Copy link
Contributor

Thanks! Merging.

@martelletto martelletto merged commit 60992d5 into Yubico:master May 18, 2020
@fmeum fmeum deleted the fhenneke_bluetooth_hid branch May 18, 2020 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants