Skip to content
This repository has been archived by the owner on Jul 27, 2022. It is now read-only.

Feature/sha512 #5

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

Feature/sha512 #5

wants to merge 11 commits into from

Conversation

mvo5
Copy link

@mvo5 mvo5 commented Jun 5, 2013

This is a request for feedback, not yet a fully ready pull request.

It adds a nodejs cli mode to get the password (prototype only for now) and updates the code to prepare for using sha512 instead of sha1 (as its much harder to crack, some numbers in http://arstechnica.com/security/2013/05/how-crackers-make-minced-meat-out-of-your-passwords/).

If you aggree with the general direction I will continue and add a option to select the cypher and use sha1 only for backport compat.

@mvo5
Copy link
Author

mvo5 commented Jun 6, 2013

The sha512 support is working on my system now.

I also added a UI to select the hash algorithm for backward compatiblity and changed the default to sha512 and len 14.

Feedback welcome and thanks for creating this extension in the first place.

@mvo5
Copy link
Author

mvo5 commented Jun 6, 2013

The node cli part needs work as the password should obviously not passed via the commandline, that is just a proof-of-concept.

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.

1 participant