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 comments) (0 reactions) (0 assignees)HTML (1.526 forks)batch import

Good first issuehelp wantedjspriority: MEDIUMtask

Métricas do repositório

Stars: (1.794 stars)
Métricas de merge de PR: (Nenhuma PRs mesclada em 30d)

Description

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

Guia do colaborador

Direção de pesquisa: Revise o uso atual de bcrypt nodejs em app/data/user dao.js para entender as diferenças de API. Em seguida, instale o pacote bcrypt e atualize o arquivo para usar os métodos hash e compare do bcrypt. Atualize package.json, package lock.json e readme.md de acordo. Teste as alterações executando npm test.
Pilha de tecnologia: javascriptnodejs
Domain: backendsecurity
Tipo Issue: Refatoração
Difficulty: 2
Tempo estimado: 1-3 horas
Status da atividade: Recente
Clarity: Claro
Prerequisites: Node.js basicsnpm package managementGit and GitHub workflow
Simpatia para novatos: 85