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

Remove CTAP 2.1 flag #252

Merged
merged 3 commits into from
Jan 8, 2021
Merged

Conversation

kaczmarczyck
Copy link
Collaborator

In our new develop branch, we drop support for CTAP 2.0 separately from CTAP 2.1.

This is related to #106 and makes #128 much less important since the amount of cfg statements is significantly reduced. (I leave it to @jmichelp if we want to close #128.)

  • Tests pass
  • Appropriate changes to README are included in PR

This is a copy of #248 and requests against the correct branch. It is based on #250 and is implemented on top of its changes.

src/ctap/mod.rs Outdated Show resolved Hide resolved
src/ctap/storage.rs Show resolved Hide resolved
@kaczmarczyck kaczmarczyck merged commit 32d5ff9 into google:develop Jan 8, 2021
@kaczmarczyck kaczmarczyck deleted the remove-flag-2-1 branch January 8, 2021 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants