Skip to content
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

ADDED PRIVATE KEY ENCRYPTION #45

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

ADDED PRIVATE KEY ENCRYPTION #45

wants to merge 4 commits into from

Conversation

adanielpincab
Copy link
Contributor

This new version improves the using and security of adresses:

  • Now addresses generated are stored in a file with .wallet extension. That makes them recognizable as simplecoin addresses by the program, so you don't have to manually enter your wallet credentials everytime you make a transaction.

  • But even more important: when creating a new address, the program gives you the option of encrypting it's private key with a password. If you want to do a transaction with an encrypted address, you will have to enter the password in order to get the private key. This makes the address safer, preventing the private key to be stolen by someone that could have access to the address file.

This new feature solves an important problem of issue #6 (Preventing someone from using other users wallets)

Added venv folder, which I use for testing
Various changes made to introduce ".wallet" files (with encryption mode)
Changed a little the encription method and removed testing addresses
@adanielpincab adanielpincab changed the title ADDED ENCRYPTION ADDED PRIVATE KEY ENCRYPTION Oct 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant