OWASP/NodeGoat

Migration to bcrypt

オープン

#158 opened on 2019/08/06

 (14 件のコメント) (0 件のリアクション) (0 人の担当者)HTML (2,792 件のフォーク)batch import
Good first issuehelp wantedjspriority: MEDIUMtask

Repository metrics

Stars
 (2,060 個のスター)
PR merge metrics
 (30d に merged PR はありません)

説明

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

コントリビューターガイド