3 ms·
Can you clarify why older passwords were not rehashed to utilize the new scheme? Or are those just passwords of people who haven't logged in since you converted
by akerl_ 13y ago
Can you clarify why older passwords were not rehashed to utilize the new scheme? Or are those just passwords of people who haven't logged in since you converted your backend to use scrypt?
- Houshalter 13y agoCouldn't you hash the old hash with the new one even if they didn't log in? I.e. hash the password with the old SHA-1 first, then bcrypt that. I'm not sure if that would work or not.
- akerl_ 13y agoIn theory you could, but then the code has to support both login procedures forever (hashing once with scrypt or hashing with SHA1 and then scrypt). That's why, if it were me, I'd likely set it up so if someone with a SHA1-hashed password logged in, it first did the SHA1 comparison, and if that succeeded the login process dropped an scrypted version into the DB.
- cmircea 13y agoYou could just always do a hash using SHA1 then feed that to bcrypt. This way you only have one code path, for all users.
- akerl_ 13y agoExcept that for some people, you only have a SHA1 hash, and for others, you only have an scrypt hash. Since they (I hope) weren't storing the plaintext (pre-hash) passwords alongside the hashed versions, the only time they can change the initial hashing mechanism is when passwords are provided to them during a password change or user login.
- ErrantX 13y agoHash all the sha1 passwords with scrypt. Hash all new passwords with sha1 then scrypt. Your new hash mechanism is sha1 then scrypt. There is no excuse to do otherwise :)
- hga 13y ago(ADJUSTED:) Yes, if when you said "hash the password" you meant the result of what they'd done originally and that was stored in their databe. They don't have a copy of the password until and unless the user logs in again. These are all "one way" or "trapdoor" functions, plaintext -> irreversible cyphertext. Irreversible in that we think mathematically you can't go back.
- deleted 13y ago[deleted]