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.

Sign up for GitHub

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

Jump to bottom

sha1 not fully secure #9

Open
ghost opened this issue Jan 7, 2014 · 5 comments
Open

sha1 not fully secure #9

ghost opened this issue Jan 7, 2014 · 5 comments

Comments

@ghost
Copy link

ghost commented Jan 7, 2014

SHA1 is not really a good option for password hashing. SHA512 should be used instead. Of course I'm happy to fork create a pull request if that's cool.
@w-vi
Copy link

w-vi commented Jan 9, 2014

Go ahead, if nobody else is interested I am :)

@adrianvalenz
Copy link

Was this ever done?

@ghost
Copy link
Author

ghost commented Mar 28, 2014

Not yet, been too busy. So someone else is still welcome to pick it up if they desire. :) It's fairly straightforward.

@mazuhl mazuhl mentioned this issue Apr 30, 2014
Open
@mazuhl
Copy link

mazuhl commented Apr 30, 2014

@CaptSaltyJack @aydreeuhn - have switched over from SHA1 to SHA512.

@ghost
Copy link
Author

ghost commented Apr 30, 2014

Nice work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mazuhl @adrianvalenz @w-vi