Merge pull request #5630 from weseek/imprv/extend-minimum-password-le…

…ngth fix: Displaying minimum length of password
weseek · Mar 31, 2022 · b584e2a · b584e2a
2 parents 0ab6197 + d2910ae
commit b584e2a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/packages/app/src/server/middlewares/register-form-validator.ts b/packages/app/src/server/middlewares/register-form-validator.ts
@@ -1,5 +1,6 @@
 import { body, validationResult } from 'express-validator';
 
+const PASSOWRD_MINIMUM_NUMBER = 8;
 // form rules
 export const registerRules = () => {
   return [
@@ -18,8 +19,8 @@ export const registerRules = () => {
     body('registerForm.password')
       .matches(/^[\x20-\x7F]*$/)
       .withMessage('Password has invalid character')
-      .isLength({ min: 6 })
-      .withMessage('Password minimum character should be more than 6 characters')
+      .isLength({ min: PASSOWRD_MINIMUM_NUMBER })
+      .withMessage('Password minimum character should be more than 8 characters')
       .not()
       .isEmpty()
       .withMessage('Password field is required'),

diff --git a/packages/app/src/server/routes/apiv3/user-activation.ts b/packages/app/src/server/routes/apiv3/user-activation.ts
@@ -3,6 +3,7 @@ import * as express from 'express';
 import { body, validationResult } from 'express-validator';
 import ErrorV3 from '../../models/vo/error-apiv3';
 
+const PASSOWRD_MINIMUM_NUMBER = 8;
 // validation rules for complete registration form
 export const completeRegistrationRules = () => {
   return [
@@ -17,8 +18,8 @@ export const completeRegistrationRules = () => {
     body('password')
       .matches(/^[\x20-\x7F]*$/)
       .withMessage('Password has invalid character')
-      .isLength({ min: 6 })
-      .withMessage('Password minimum character should be more than 6 characters')
+      .isLength({ min: PASSOWRD_MINIMUM_NUMBER })
+      .withMessage('Password minimum character should be more than 8 characters')
       .not()
       .isEmpty()
       .withMessage('Password field is required'),