Migration to bcrypt · OWASP/NodeGoat#158

2019-08-06T06:40:09.000Z

### Context - This is part of `release-1.5` #148 - MEDIUM priority task ### Tasks - [ ] Remove dependency `bcrypt-nodejs` in `package.json` - [ ] Add dependency `bcrypt` in `package.json` - [ ] Migrate file [app/data/user-dao.js](https://github.com/OWASP/NodeGoat/blob/master/app/data/user-dao.js) to bcrypt - [ ] Validate the instalation with the local test - [ ] Add and submit the changes in `package-lock.json` - [ ] Add the primary dependency list to the `readme.md` - [ ] Check that the npm tasks are working as expected - [ ] Update the `readme.md` with the extra relevant info (if needed) ### Assignation - This tasks is open for assignation, just claim it (as reply to this) and submit your PR ;-) ### Important - Check the current roadmap #148 - Don't forget to check the [Contributing Guidelines](https://github.com/OWASP/NodeGoat/blob/master/CONTRIBUTING.md) and follow the [Code of Conduct](https://github.com/OWASP/NodeGoat/blob/master/CONTRIBUTING.md)

(14 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)HTML (1.794 Stars) (1.526 Forks)batch import

Good first issuehelp wantedjspriority: MEDIUMtask

Beschreibung

Context

This is part of release-1.5 #148
MEDIUM priority task

Tasks

Remove dependency bcrypt-nodejs in package.json
Add dependency bcrypt in package.json
Migrate file app/data/user-dao.js to bcrypt
Validate the instalation with the local test
Add and submit the changes in package-lock.json
Add the primary dependency list to the readme.md
Check that the npm tasks are working as expected
Update the readme.md with the extra relevant info (if needed)

Assignation

This tasks is open for assignation, just claim it (as reply to this) and submit your PR ;-)

Important

Check the current roadmap #148
Don't forget to check the Contributing Guidelines and follow the Code of Conduct

Contributor Guide

Tech Stack: javascriptnodejs
Domain: backendsecurity
Issue Type: refactor
Schwierigkeit: 2
Geschätzte Zeit: 1-3 hours
Aktivitätsstatus: fresh
Klarheit: clear
Voraussetzungen: Node.js basicsnpm package managementGit and GitHub workflow
Einsteigerfreundlichkeit: 85
Research-Richtung: Review the current usage of bcrypt nodejs in app/data/user dao.js to understand the API differences. Then install the bcrypt package and update the file to use bcrypt's hash and compare methods. Update package.json, package lock.json, and readme.md accordingly. Test the changes by running npm test.