Skip to content
This repository has been archived by the owner on Mar 22, 2024. It is now read-only.

User Authentication (Backend) #2

Closed
Decode84 opened this issue Mar 22, 2022 · 1 comment · Fixed by #32
Closed

User Authentication (Backend) #2

Decode84 opened this issue Mar 22, 2022 · 1 comment · Fixed by #32
Assignees
Labels
Backend Node, Express, Databases & other nonvisual elements Priority high Very important and should come first if possible Task hard Takes much knowledge and time to do

Comments

@Decode84
Copy link
Owner

User authentication, skal inkludere hashing af password, hvor der skal gøres brug af bcrypt. Som også skal verificere det indtastet password. Derudover skal der også laves check på om username/email findes i databasen.

Derudover skal der valideres på det indtastet input, så som

  • 8 min length
@Murmeldyret Murmeldyret changed the title User Authentication User Authentication (Backend) Mar 24, 2022
@Murmeldyret Murmeldyret added Backend Node, Express, Databases & other nonvisual elements Priority high Very important and should come first if possible Task hard Takes much knowledge and time to do labels Mar 24, 2022
@Decode84
Copy link
Owner Author

Decode84 commented Mar 24, 2022

Evt benyt én af disse to til at hash passwords med.

  • Bcrypt: Gammel. Men virker (Mener den er en standard lige pt.)
  • Argon2: Ny, moderne. (Men kender ikke så meget til den. Mange nyere projekter er vist ved at skifte over til den.)

https://github.com/ranisalt/node-argon2
https://github.com/kelektiv/node.bcrypt.js

@Murmeldyret Murmeldyret added this to the Minimal Viable Product milestone Mar 24, 2022
@Aavild Aavild pinned this issue Mar 24, 2022
@Aavild Aavild self-assigned this Mar 25, 2022
@Decode84 Decode84 linked a pull request Mar 27, 2022 that will close this issue
8 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Backend Node, Express, Databases & other nonvisual elements Priority high Very important and should come first if possible Task hard Takes much knowledge and time to do
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants