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

Do not return NO_CREDENTIALS if there are no RKs and meta is requested #404

Merged
merged 2 commits into from
Mar 27, 2020

Conversation

rgerganov
Copy link

Fixes-issue: #403

Radoslav Gerganov added 2 commits March 26, 2020 15:14
This patch fixes the following issues to make Google Chrome happy:
1. Adds CTAP_CBOR_CRED_MGMT(0x0A) which is an alias to CTAP_CBOR_CRED_MGMT_PRE(0x41)
2. Returns success instead of NO_CREDENTIALS when there are no RKs
3. Skip the "icon" property if it's empty

Tested with Google Chrome Version 80.0.3987.149
@rgerganov
Copy link
Author

I have added one more patch which add support for the security manager in Google Chrome

@conorpp
Copy link
Member

conorpp commented Mar 27, 2020

Thank you!!

@conorpp conorpp merged commit f002d08 into solokeys:master Mar 27, 2020
@My1
Copy link
Contributor

My1 commented Mar 27, 2020

nice to see this joined

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.

3 participants