EPMRPP-100064 || Exclude spaces from password validation

Author: BlazarQSO

app/src/common/utils/validation/validate.js

@@ -35,8 +35,7 @@ export const login = composeValidators([isNotEmpty, regex(/^[0-9a-zA-Z-_.]{1,128
 export const oldPassword = composeValidators([isNotEmpty, regex(/^(.){4,256}$/)]);
 export const password = composeValidators([
   isNotEmpty,
-  minLength(8),
-  regex(/^(?=.*[a-z])(?=.*[A-Z])(?=.*\d)(?=.*[\W_]).+$/),
+  regex(/^(?=.*\d)(?=.*[A-Z])(?=.*[a-z])(?=.*[^\w\d\s:])([^\s]){8,256}$/),
 ]);
 export const userName = composeValidators([
   isNotEmpty,

app/src/pages/admin/allUsersPage/usersToolbar/actionPanel/actionPanel.jsx

@@ -121,7 +121,7 @@ export class ActionPanel extends Component {
         email: values.email,
         fullName: values.fullName,
         login: values.login,
-        password: values.password.trim(),
+        password: values.password,
         projectRole: values.projectRole,
       },
     })

app/src/pages/inside/profilePage/modals/changePasswordModal/changePasswordModal.jsx

@@ -72,9 +72,9 @@ const messages = defineMessages({
 @reduxForm({
   form: 'changePasswordForm',
   validate: ({ oldPassword, newPassword, confirmPassword }) => ({
-    oldPassword: commonValidators.oldPassword(oldPassword.trim()),
-    newPassword: commonValidators.password(newPassword.trim()),
-    confirmPassword: newPassword.trim() !== confirmPassword.trim() && 'profileConfirmPassword',
+    oldPassword: commonValidators.oldPassword(oldPassword),
+    newPassword: commonValidators.password(newPassword),
+    confirmPassword: newPassword !== confirmPassword && 'profileConfirmPassword',
   }),
 })
 @track()

app/src/pages/inside/profilePage/personalInfoBlock/personalInfoBlock.jsx

@@ -165,7 +165,7 @@ export class PersonalInfoBlock extends Component {
   changePasswordHandler = (data) => {
     fetch(URLS.userChangePassword(), {
       method: 'post',
-      data: { oldPassword: data.oldPassword.trim(), newPassword: data.newPassword.trim() },
+      data: { oldPassword: data.oldPassword, newPassword: data.newPassword },
     })
       .then(() => {
         this.props.showNotification({

app/src/pages/outside/loginPage/pageBlocks/changePasswordBlock/changePasswordForm/changePasswordForm.jsx

@@ -74,9 +74,8 @@ const hints = defineMessages({
 @reduxForm({
   form: 'changePassword',
   validate: ({ password, passwordRepeat }) => ({
-    password: commonValidators.password(password.trim()),
-    passwordRepeat:
-      (!passwordRepeat || passwordRepeat.trim() !== password.trim()) && 'confirmPasswordHint',
+    password: commonValidators.password(password),
+    passwordRepeat: (!passwordRepeat || passwordRepeat !== password) && 'confirmPasswordHint',
   }),
 })
 @injectIntl
@@ -102,7 +101,7 @@ export class ChangePasswordForm extends PureComponent {
     fetch(URLS.userPasswordReset(), {
       method: 'post',
       data: {
-        password: password.trim(),
+        password,
         uuid,
       },
     })

app/src/pages/outside/registrationPage/registrationForm/registrationForm.jsx

@@ -62,9 +62,8 @@ const messages = defineMessages({
 @reduxForm({
   form: 'registration',
   validate: ({ login, name, password, confirmPassword }) => ({
-    password: commonValidators.password(password.trim()),
-    confirmPassword:
-      (!confirmPassword || confirmPassword.trim() !== password.trim()) && 'confirmPasswordHint',
+    password: commonValidators.password(password),
+    confirmPassword: (!confirmPassword || confirmPassword !== password) && 'confirmPasswordHint',
     login: commonValidators.login(login),
     name: commonValidators.userName(name),
   }),

app/src/pages/outside/registrationPage/registrationPageContainer.jsx

@@ -89,11 +89,11 @@ export class RegistrationPageContainer extends Component {
     const data = {
       fullName: name,
       login,
-      password: password.trim(),
+      password,
       email,
     };
     return fetch(URLS.userRegistration(), { method: 'post', data, params: { uuid } })
-      .then(() => this.props.loginAction({ login, password: password.trim() }))
+      .then(() => this.props.loginAction({ login, password }))
       .catch(({ message }) => {
         this.props.showNotification({
           type: NOTIFICATION_TYPES.ERROR,