Improve handling inputs globally

Author: kkamara

src/models/v1/product.js

@@ -645,7 +645,7 @@ module.exports = (sequelize, DataTypes) => {
         return "The name field is required.";
       } else if ("string" !== typeof payload.name) {
         return "The name field must be a string.";
-      } else if (50 < payload.name.length) {
+      } else if (50 < payload.name.trim().length) {
         return "The name field must be less than 51 characters.";
       } else {
         const nameExists = await this.getProductBySlug(
@@ -666,18 +666,22 @@ module.exports = (sequelize, DataTypes) => {
         return "The weight field is required.";
       } else if (null === `${payload.weight}`.match(numberWithOptionalDecimalPartRegex)) {
         return "The weight field must be a number"
+      } else if (500 < Number(payload.weight)) {
+        return "The weight field number must not be greater than 500.";
       }
 
       if (undefined === payload.price) {
         return "The price field is required.";
       } else if (null === `${payload.price}`.match(numberWithOptionalDecimalPartRegex)) {
         return "The price field must be a number"
+      } else if (500 < Number(payload.price)) {
+        return "The price field number must not be greater than 500.";
       }
 
       if (payload.description) {
         if ("string" !== typeof payload.description) {
           return "The description field must be a string.";
-        } else if (1000 < payload.description.length) {
+        } else if (1000 < payload.description.trim().length) {
           return "The description field must be less than 1001 characters.";
         }
       }
@@ -728,19 +732,19 @@ module.exports = (sequelize, DataTypes) => {
       if (payload.stripeProductId) {
         if ("string" !== typeof payload.stripeProductId) {
           return "The stripe product id field must be of type string."
-        } else if (15 > payload.stripeProductId.length) {
+        } else if (15 > payload.stripeProductId.trim().length) {
           return "The stripe product id field length must be greater than 15 characters.";
-        } else if (30 < payload.stripeProductId.length) {
+        } else if (30 < payload.stripeProductId.trim().length) {
           return "The stripe product id field length must not exceed 30 characters.";
         }
       }
 
       if (payload.stripePriceId) {
         if ("string" !== typeof payload.stripePriceId) {
           return "The stripe price id field must be of type string."
-        } else if (15 > payload.stripePriceId.length) {
+        } else if (15 > payload.stripePriceId.trim().length) {
           return "The stripe price id field length must be greater than 15 characters.";
-        } else if (50 < payload.stripePriceId.length) {
+        } else if (50 < payload.stripePriceId.trim().length) {
           return "The stripe price id field length must not exceed 50 characters.";
         }
       }
@@ -766,7 +770,7 @@ module.exports = (sequelize, DataTypes) => {
         return "The name field is required.";
       } else if ("string" !== typeof payload.name) {
         return "The name field must be a string.";
-      } else if (50 < payload.name.length) {
+      } else if (50 < payload.name.trim().length) {
         return "The name field must be less than 51 characters.";
       } else {
         const newSlug = slugify(payload.name);
@@ -790,18 +794,22 @@ module.exports = (sequelize, DataTypes) => {
         return "The weight field is required.";
       } else if (null === `${payload.weight}`.match(numberWithOptionalDecimalPartRegex)) {
         return "The weight field must be a number"
+      } else if (500 < Number(payload.weight)) {
+        return "The weight field number must not be greater than 500.";
       }
 
       if (undefined === payload.price) {
         return "The price field is required.";
       } else if (null === `${payload.price}`.match(numberWithOptionalDecimalPartRegex)) {
         return "The price field must be a number"
+      } else if (500 < Number(payload.price)) {
+        return "The price field number must not be greater than 500.";
       }
 
       if (payload.description) {
         if ("string" !== typeof payload.description) {
           return "The description field must be a string.";
-        } else if (1000 < payload.description.length) {
+        } else if (1000 < payload.description.trim().length) {
           return "The description field must be less than 1001 characters.";
         }
       }
@@ -883,14 +891,14 @@ module.exports = (sequelize, DataTypes) => {
 
     static getNewProductData(payload) {
       const result = {
-        name: payload.name,
+        name: payload.name.trim(),
         units: payload.units,
         weight: payload.weight,
         price: payload.price,
         isLive: false,
       };
       if (payload.description) {
-        result.description = payload.description;
+        result.description = payload.description.trim();
       }
       if (payload.category) {
         result.category = payload.category;
@@ -904,24 +912,24 @@ module.exports = (sequelize, DataTypes) => {
         }
       }
       if (payload.stripeProductId) {
-        result.stripeProductId = payload.stripeProductId;
+        result.stripeProductId = payload.stripeProductId.trim();
       }
       if (payload.stripePriceId) {
-        result.stripePriceId = payload.stripePriceId;
+        result.stripePriceId = payload.stripePriceId.trim();
       }
       return result;
     }
 
     static getEditProductData(payload) {
       const result = {
-        name: payload.name,
+        name: payload.name.trim(),
         units: payload.units,
         weight: payload.weight,
         price: payload.price,
         isLive: false,
       };
       if (payload.description) {
-        result.description = payload.description;
+        result.description = payload.description.trim();
       }
       if (payload.category) {
         result.category = payload.category;
@@ -935,10 +943,10 @@ module.exports = (sequelize, DataTypes) => {
         }
       }
       if (payload.stripeProductId) {
-        result.stripeProductId = payload.stripeProductId;
+        result.stripeProductId = payload.stripeProductId.trim();
       }
       if (payload.stripePriceId) {
-        result.stripePriceId = payload.stripePriceId;
+        result.stripePriceId = payload.stripePriceId.trim();
       }
       return result;
     }

src/models/v1/user.js

@@ -292,7 +292,7 @@ module.exports = (sequelize, DataTypes) => {
         return "The email field is missing.";
       } else if ("string" !== typeof bodyInput.email) {
         return "The email field must be of type string";
-      } else if (30 < bodyInput.email.length) {
+      } else if (30 < bodyInput.email.trim().length) {
         return "The email field length must not exceed 30 characters.";
       } else if (null === bodyInput.email.match(validEmailRegex)) {
         return "The email field must be a valid email address.";
@@ -508,7 +508,7 @@ module.exports = (sequelize, DataTypes) => {
         return "The email field is missing.";
       } else if ("string" !== typeof bodyInput.email) {
         return "The email field must be of type string";
-      } else if (30 < bodyInput.email.length) {
+      } else if (30 < bodyInput.email.trim().length) {
         return "The email field length must not exceed 30 characters.";
       } else if (null === bodyInput.email.match(validEmailRegex)) {
         return "The email field must be a valid email address.";
@@ -541,7 +541,7 @@ module.exports = (sequelize, DataTypes) => {
         return false;
       }
 
-      return { email, password, };
+      return { email: email.trim(), password, };
     }
 
     /**

src/models/v1/userAddress.js

@@ -153,36 +153,46 @@ module.exports = (sequelize, DataTypes) => {
         return "The address line 1 field is required.";
       } else if ("string" !== typeof payload.addressLine1) {
         return "The address line 1 field must be of type string.";
+      } else if (50 < payload.addressLine1.trim().length) {
+        return "The address line 1 field must not exceed 50 characters";
       }
 
       if (payload.addressLine2) {
         if ("string" !== typeof payload.addressLine2) {
           return "The address line 2 field must be of type string.";
         }
+      } else if (50 < payload.addressLine2.trim().length) {
+        return "The address line 2 field must not exceed 50 characters";
       }
 
       if (!payload.zipCode) {
         return "The zip code field is required.";
       } else if ("string" !== typeof payload.zipCode) {
         return "The zip code field must be of type string.";
+      } else if (10 < payload.zipCode.trim().length) {
+        return "The zip code field must not exceed 10 characters";
       }
 
       if (!payload.city) {
         return "The city field is required.";
       } else if ("string" !== typeof payload.city) {
         return "The city field must be of type string.";
+      } else if (20 < payload.city.trim().length) {
+        return "The city field must not exceed 20 characters";
       }
 
       if (!payload.state) {
         return "The state field is required.";
       } else if ("string" !== typeof payload.state) {
         return "The state field must be of type string.";
+      } else if (20 < payload.state.trim().length) {
+        return "The address line 1 field must not exceed 20 characters";
       }
 
       if (payload.telephoneAreaCode) {
         if ("string" !== typeof payload.telephoneAreaCode) {
           return "The telephone area code field must be of type string.";
-        } else if (6 < payload.telephoneAreaCode.length) {
+        } else if (6 < payload.telephoneAreaCode.trim().length) {
           return "The telephone area code field must not be greater than 6 characters.";
         }
       }
@@ -191,7 +201,7 @@ module.exports = (sequelize, DataTypes) => {
         return "The telephone field is required.";
       } else if ("string" !== typeof payload.telephone) {
         return "The telephone field must be of type string.";
-      } else if (30 < payload.telephone.length) {
+      } else if (30 < payload.telephone.trim().length) {
         return "The telephone field must not be greater than 30 characters.";
       }
 
@@ -216,31 +226,31 @@ module.exports = (sequelize, DataTypes) => {
       const result = {};
 
       if (payload.addressLine1) {
-        result.addressLine1 = payload.addressLine1;
+        result.addressLine1 = payload.addressLine1.trim();
       }
 
       if (payload.addressLine2) {
-        result.addressLine2 = payload.addressLine2;
+        result.addressLine2 = payload.addressLine2.trim();
       }
 
       if (payload.zipCode) {
-        result.zipCode = payload.zipCode;
+        result.zipCode = payload.zipCode.trim();
       }
 
       if (payload.city) {
-        result.city = payload.city;
+        result.city = payload.city.trim();
       }
 
       if (payload.state) {
-        result.state = payload.state;
+        result.state = payload.state.trim();
       }
 
       if (payload.telephoneAreaCode) {
-        result.telephoneAreaCode = payload.telephoneAreaCode;
+        result.telephoneAreaCode = payload.telephoneAreaCode.trim();
       }
 
       if (payload.telephone) {
-        result.telephone = payload.telephone;
+        result.telephone = payload.telephone.trim();
       }
 
       return result;