Skip to content

Commit d44aace

Browse files
Merge pull request #4 from arkadiyt/bcrypt-fix
Fix bcrypt checklist item
2 parents 2057ede + 8ee723d commit d44aace

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

security-checklist.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66
##### AUTHENTICATION SYSTEMS (Signup/Signin/2 Factor/Password reset)
77
- [ ] Use HTTPS everywhere.
8-
- [ ] Store password hashes using `Bcrypt` with a random `salt`.
8+
- [ ] Store password hashes using `Bcrypt` (no salt necessary - `Bcrypt` does it for you).
99
- [ ] Destroy the session identifier after `logout`.
1010
- [ ] Destroy all active sessions on reset password (or offer to).
1111
- [ ] Must have the `state` parameter in OAuth2

0 commit comments

Comments
 (0)