Hacker News new | ask | show | jobs
by jstanley 3043 days ago
This is cool, but doesn't seem to work properly.

I tried it with the passwords "password", "test", "123", "12345" and it didn't report any of them as compromised, which they surely are.

EDIT: It's because it's including a trailing endline before hashing every password. This is a serious bug.

EDIT2: Latest version in the repo does work.

1 comments

Oh wow. Can't believe I missed that. Thank you so much! Will fix right away.
I've sent a pull request.

EDIT: Which you've now merged :)