-
Notifications
You must be signed in to change notification settings - Fork 5
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
Reverse HOTP #1
Closed
Closed
Reverse HOTP #1
Conversation
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
Let's consider this being ready to review. The remaining tasks' results will be supplied in separate PRs. |
Note: confirmed to work in the NK3's embedded runner (
CLI is WIP under: |
sosthene-nitrokey
requested changes
Nov 10, 2022
5 tasks
Support CTAPHID transport. This is realized by sending CCID traffic into CTAP vendor command. Fixes #2
szszszsz
force-pushed
the
reverse-hotp
branch
from
November 18, 2022 13:16
8479314
to
fd15486
Compare
Updated as suggested in the review. Remaining tasks moved to separate tickets. |
New command is added: VerifyCode This is used e.g. in Heads, to execute the Measured Boot operation.
To avoid desynchronization between the host's and device's counters
This way different operations cannot be mixed on the same credential
szszszsz
force-pushed
the
reverse-hotp
branch
from
November 18, 2022 13:43
ba5359b
to
04b2ebb
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Add the reverse HOTP feature
Tasks
verify_code
, instead of the currently stringDescription
Verify the HOTP code coming from a PC host, and show visually to user, that the code is correct or not, with a green or red LED respectively. Does not need authorization by design.
See more at:
Solution contains means to avoid desynchronization between the host's and device's counters. Device calculates up to 9 values ahead of its current counter to find the matching code (in total it calculates HOTP code for 10 subsequent counter positions). In case:
Device will stop verifying the HOTP codes in case, when the difference between the host and on-device counters will be greater or equal to 10.
cc @sosthene-nitrokey