This repository was archived by the owner on Jun 17, 2021. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 274
V6 Removing native js dependencies #259
Merged
holgerd77
merged 15 commits into
ethereumjs:v6
from
Tenderly:v6-removing-native-js-dependencies
Jul 15, 2020
Merged
Changes from all commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
4234861
Remove keccak native dependency.
nebojsa94 e17e174
Remove secp256k1 native dependency.
nebojsa94 498fa1c
Use secp256k1 lib instead of shims.
nebojsa94 79d7a2b
Wrapper around secp256k1 to support v3 API.
nebojsa94 91bdfce
Add missing function.
nebojsa94 60c83ad
Add missing function.
nebojsa94 04a41cf
Secp256k1 wrapper refactor and testing.
nebojsa94 b91d513
Fix ecdhUnsafe function.
nebojsa94 c7124c7
Fix browser tests.
nebojsa94 d51ba66
Bump test coverage.
nebojsa94 ed84154
Handle invalid length of private key during validation.
nebojsa94 02bc4c5
Add elliptic package, file renaming and minor fixes.
nebojsa94 0423e4f
Switch to using secp256k1 adapter.
nebojsa94 3876bde
Require secp256k1 from ethereum-cryptography.
nebojsa94 71ccac5
Add comments to secp256k1 wrapper.
nebojsa94 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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.
Note for reviewers: There aren't versions of keccak other than 224, 256, 384, and 512. This API is confusing, but we have to keep it for backward compatibility reasons.