-
Notifications
You must be signed in to change notification settings - Fork 50
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
sphincs+ support, for post-quantum crypto #169
Closed
+1,013
−24
Closed
Changes from all commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
6766f12
Added SPX test cases (copy-pasted from ed25510 test cases)
cryptojedi caa2ca9
Added support for SPX signatures, still completely untested
cryptojedi 4b30a78
Changed from PyNaCl API to simpler (byte-string based) pyspx API
cryptojedi 59b33e6
Added required spx file
cryptojedi b49d6dc
Fix import of specific SPX instance
joostrijneveld 641fb65
Remove nacl-specific Exception for PySPX
joostrijneveld 451d8cc
Add PySPX as a requirement
joostrijneveld 5eb3cda
Fix tests for SPHINCS+
joostrijneveld 0f56fbe
Fix missing exception handling for invalid schemes
joostrijneveld ea6c942
Clarify silenced Exception, mark for code coverage
joostrijneveld 7f52678
Fixed merge conflict
cryptojedi d653691
Fix small copypasting errors
joostrijneveld 9beace8
Restructure pyspx conditional import
lukpueh 5458f62
Update pinned spx dependency in dev requirements
lukpueh 154bee1
Add latest pyspx as optional dependency
lukpueh 335199d
Add Sphincs+/PySPX infos to README
lukpueh a67a6b3
Remove pubkey arg from spx_keys.create_signature
lukpueh 44db3df
Add minor style/clean-up fixes in spx related code
lukpueh 2c453c1
Fix error message in spx create_signature
lukpueh b0735d5
Adopt data encoding in spx sig creation function
lukpueh beab2a1
Fix spx_keys.verify_signature doctest
lukpueh f6e47b9
Remove duplicate doctest lines
lukpueh 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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,6 @@ | ||
cryptography | ||
pynacl | ||
pyspx | ||
tox | ||
coverage | ||
coveralls | ||
|
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,6 @@ | ||
cryptography==2.3.1 | ||
pynacl==1.2.1 | ||
pyspx==0.4.0 | ||
six==1.11.0 | ||
tox==3.2.1 | ||
coveralls==1.3.0 | ||
|
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
cryptography | ||
pynacl | ||
pyspx | ||
six | ||
colorama |
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
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
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.
Should there be a check to see if a provided
filepath
is a directory?And it could either be a warning or it could accept the directory and change the path to
directory/keyid
.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.
Good point. I created an issue for this. Feel free to comment there. #172