Skip to content

Commit

Permalink
Merge pull request #1640 from dbauszus-glx/replace-bcrypt-module
Browse files Browse the repository at this point in the history
Replace bcrypt node module with utils module
  • Loading branch information
RobAndrewHurst authored Nov 4, 2024
2 parents ec49bfb + 7e82cbf commit 04457f1
Show file tree
Hide file tree
Showing 5 changed files with 1,224 additions and 7 deletions.
2 changes: 0 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -54,8 +54,6 @@ Dynamic module imports reduce the need to bundle 3rd party libraries such as [Ta

Node.js v18+

[bcrypt](https://www.npmjs.com/package/bcrypt) - A library to help you hash passwords.

[jsonwebtoken](https://www.npmjs.com/package/jsonwebtoken) - A Node implementation of JSON Web Token.

[Node-Postgres](https://github.com/brianc/node-postgres) - PostgreSQL client for Node.
Expand Down
4 changes: 2 additions & 2 deletions mod/user/fromACL.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,13 +7,13 @@ This module exports the fromACL method to request and validate a user from the A
@requires module:/utils/reqHost
@requires module:/utils/mailer
@requires module:/utils/languageTemplates
@requires bcrypt
@requires module:/utils/bcrypt
@requires crypto
@module /user/fromACL
*/

const bcrypt = require('bcrypt')
const bcrypt = require('../utils/bcrypt')

const crypto = require('crypto')

Expand Down
4 changes: 2 additions & 2 deletions mod/user/register.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,13 @@ Exports the [user] register method for the /api/user/register route.
@requires /utils/reqHost
@requires /utils/mailer
@requires /utils/languageTemplates
@requires bcrypt
@requires /utils/bcrypt
@requires crypto
@module /user/register
*/

const bcrypt = require('bcrypt')
const bcrypt = require('../utils/bcrypt')

const crypto = require('crypto')

Expand Down
Loading

0 comments on commit 04457f1

Please sign in to comment.