Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace bcrypt node module with utils module #1640

Merged
merged 9 commits into from
Nov 4, 2024
Merged
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
Loading