Skip to content
This repository has been archived by the owner on Nov 21, 2019. It is now read-only.

Validate manual private key #1671

Merged
merged 8 commits into from
Mar 13, 2018
Merged

Validate manual private key #1671

merged 8 commits into from
Mar 13, 2018

Conversation

gamalielhere
Copy link
Collaborator

@gamalielhere
Copy link
Collaborator Author

Hey @kvhnuke and @SteveMieskoski , if you can, can you take a look at this and see if it's fine. Just need a 2nd pair of eyes to make sure.

@kvhnuke
Copy link
Collaborator

kvhnuke commented Mar 12, 2018

I think itll be better to use https://github.com/ethereumjs/ethereumjs-util/blob/master/docs/index.md#isvalidprivate

@gamalielhere gamalielhere changed the title Add a finally clause that checks if private key is valid Validate private key before anything Mar 13, 2018
@gamalielhere gamalielhere changed the title Validate private key before anything Validate manual private key Mar 13, 2018
@gamalielhere gamalielhere merged commit 8d7b66c into mercury Mar 13, 2018
@gamalielhere gamalielhere deleted the bug/1664-issue branch March 13, 2018 23:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants