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 full Yubico OATH AID #3

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

abrasive
Copy link

@abrasive abrasive commented Jul 4, 2022

Some clients, like Yubico Authenticator on Android, use the full AID for the OATH app, while the Trussed impl is one byte shorter and so isn't currently recognised.

See for example the Yubico Authenticator source.

This change just bumps the AID out per your original comment. I have to apologise, I'm not in a position to test it myself (still waiting on a solo2 hacker key).

The full AID is used by eg. Yubico Authenticator on Android, which
previously would not recognise the Trussed OATH implementation.
@abrasive
Copy link
Author

abrasive commented Jul 4, 2022

OK, I realised I mixed up my keys and I do have a hacker one! Verified to work with both solo2-cli and Yubico Authenticator.

@michaelblyons
Copy link

🦗

@Fang-Shadow
Copy link

Bump

@mkopec
Copy link

mkopec commented Dec 4, 2024

Bump, it would be great to have this integrated finally, seeing as there is no alternative authenticator app for mobile

@michaelblyons
Copy link

Yo, Nick and friends. You still there?

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

Successfully merging this pull request may close these issues.

4 participants