-
Notifications
You must be signed in to change notification settings - Fork 70
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
Implement all commands of Ctap2 #154
Merged
Merged
Changes from all commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
bfe6fb4
WIP: Add part MakeCredentials
0e746fb
Add test for (and do some fixes) for MakeCredential CTAP1 mapping
c35619e
Adjust tests with RelyingParty and name-field
26f231a
Fix serialization of MakeCredentialOptions (individually serialize me…
f7dd900
Temporarily parse credential_public_key just as a byte-vector, until …
bf12cdb
Add Option Unparsed to AttestationStatement, to currently skip parsin…
4345004
Add commands GetAssertion and GetAssertionNext. Still missing tests.
d62a52a
WIP: Add clientpin command with all the crypto functionality commente…
15eb9ed
Remove duplicate Pin-definition
2980fb8
MakeCredentials and GetAssertion now do not store Pin but PinAuth, wh…
5f3930c
MakeCredentials: Remove pointless alias for HIDError
1b085b2
GetInfo: Fix wrong parsing of response. Forgot to strip away the stat…
e206255
Make naming of CommandError variants consistent and remove redundant …
a147e50
Replace custom error with invalid_length (even though the usage is a …
386d616
Use From<CommandError>-trait of HIDError to avoid verbose map_err-cal…
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if we use sha2 0.9, it uses digest 0.9, which in turn uses generic-array 0.13, which supports has
impl Into<[T; N]>
for GenericArrays.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have tried to use the versions that are already vendored in Firefox, to avoid update procedures there that might cause problems. Currently, Firefox has sha2 in version 0.8.2.
If you think it's not a problem to bump that dependency in Firefox as well, I can go with 0.9.