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 评论) (0 反应) (0 负责人)HTML (1,794 star) (1,526 fork)batch import

Good first issuehelp wantedjspriority: MEDIUMtask

描述

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

贡献者指南

技术栈: javascriptnodejs
领域: backendsecurity
议题类型: refactor
难度: 2
预计时间: 1-3 hours
活动状态: fresh
清晰度: clear
前置要求: Node.js basicsnpm package managementGit and GitHub workflow
新手友好度: 85
研究方向: 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.