ac-sanitizer 4.2.0 → 5.0.0

package/CHANGELOG.md

@@ -1,4 +1,34 @@
  
+# [5.0.0](https://github.com/mmpro/ac-sanitizer/compare/v4.2.1..v5.0.0) (2025-05-21 15:57:54)
+
+
+### Bug Fix
+
+* **App:** Do not require minLength of 2 chars for strings | MP | [ff2c0d4c453725cfe3f6ecd84ca5160bd907b9b1](https://github.com/mmpro/ac-sanitizer/commit/ff2c0d4c453725cfe3f6ecd84ca5160bd907b9b1)    
+If you want to set minLength, you can still do so, but default for string is 0 = no minLength  
+Related issues:
+### Tests
+
+* **App:** Fixed test | MP | [0d335b956b012023bbe6f3214d2d961ff8baa04a](https://github.com/mmpro/ac-sanitizer/commit/0d335b956b012023bbe6f3214d2d961ff8baa04a)    
+Fixed test  
+Related issues:
+## BREAKING CHANGES
+* **App:** MinLength of 2 chars for strings is no longer hardcoded/required
+
+## [4.2.1](https://github.com/mmpro/ac-sanitizer/compare/v4.2.0..v4.2.1) (2025-05-19 06:12:14)
+
+
+### Bug Fix
+
+* **App:** Fixed and improved valueType with array | MP | [0bfa6135e4588a5f24a11ab2a3024d608fe8aa21](https://github.com/mmpro/ac-sanitizer/commit/0bfa6135e4588a5f24a11ab2a3024d608fe8aa21)    
+If you set valueType for an array, we will now check all properties and return an error is there is any issue.  
+Related issues:
+### Chores
+
+* **App:** Updated packages | MP | [699b8a2a7a4d41cc7f58318e400937dfba21cf30](https://github.com/mmpro/ac-sanitizer/commit/699b8a2a7a4d41cc7f58318e400937dfba21cf30)    
+Updated packages  
+Related issues:
+ 
 # [4.2.0](https://github.com/mmpro/ac-sanitizer/compare/v4.1.4..v4.2.0) (2025-05-09 07:40:18)
 
 

package/index.js

@@ -69,7 +69,7 @@ const sanitizer = function() {
     _.some(fields, (field) => {
       // FIELD definitions
       let fieldName = field.field
-      let minLength = _.isNumber(field.minLength) ? field.minLength : 2
+      let minLength = _.isNumber(field.minLength) ? field.minLength : 0
       let allowedValues = _.get(field, 'enum', _.get(field, 'isMemberOf.group'))
       if (_.isString(allowedValues)) {
         // placeholder for enum:
@@ -274,9 +274,15 @@ const sanitizer = function() {
         else if (field.valueType) {
           // very value of the array must be of this type
           _.every(value, (v, index, value) => {
+            const { valueType, type, ...fieldProps } = field
+
             const fieldsToCheck = {
               params: {},
-              fields: [{ field: fieldName, type: _.get(field, 'valueType'), minLength: _.get(field, 'minLength'), properties: _.get(field, 'properties'), wildcardAllowed: _.get(field, 'wildcardAllowed') }]
+              fields: [{ 
+                field: fieldName, 
+                type: valueType, 
+                ...fieldProps
+              }]
             }
             _.set(fieldsToCheck, `params.${fieldName}`, v)
             const check = checkAndSanitizeValues(fieldsToCheck)

package/package.json

@@ -4,7 +4,7 @@
   "author": "Mark Poepping (https://www.admiralcloud.com)",
   "license": "MIT",
   "repository": "admiralcloud/ac-sanitizer",
-  "version": "4.2.0",
+  "version": "5.0.0",
   "homepage": "https://www.admiralcloud.com",
   "dependencies": {
     "ac-countrylist": "^1.0.13",
@@ -19,8 +19,8 @@
   "devDependencies": {
     "ac-semantic-release": "^0.4.6",
     "c8": "^10.1.3",
-    "eslint": "^9.26.0",
-    "mocha": "^11.2.2"
+    "eslint": "^9.27.0",
+    "mocha": "^11.3.0"
   },
   "scripts": {
     "test": "mocha --reporter spec --bail",

package/test/tests/array.js

@@ -37,9 +37,11 @@ module.exports = {
         value: [{ p1: 'isAString', p2: 'shouldBeRemoved' }, { p1: 'isAString2', p2: 'shouldBeRemoved2' }],
         expected: [{ p1: 'isAString' },  { p1: 'isAString2' }]
       },
-      { name: 'Array of strings with enum - uppercase vs lowercase', type: 'array', valueType: 'string', value: ['ABC'], enum: ['abc'], error: 'array_notAnAllowedValue' },
-      { name: 'Array of strings with enum - uppercase vs lowercase - with ignoreCase', type: 'array', valueType: 'string', ignoreCase: true, value: ['ABC', 'DEF'], enum: ['abc'], expected: ['ABC'] },
-    ]
+      { name: 'Array of strings with enum - uppercase vs lowercase', type: 'array', valueType: 'string', value: ['ABC'], enum: ['abc'], error: 'array_atLeastOneValueFailed' },
+      { name: 'Array of strings with enum - uppercase vs lowercase - with ignoreCase', type: 'array', valueType: 'string', ignoreCase: true, value: ['ABC', 'DEF'], enum: ['abc'], error: 'array_atLeastOneValueFailed' },
+      { name: 'Array of urls with https', type: 'array', valueType: 'url', value: ['https://www.admiralcloud.com'], expected: ['https://www.admiralcloud.com'] },
+      { name: 'Array of urls with http and https - only https is allowed', type: 'array', valueType: 'url', protocols: ['https'], value: ['http://www.admiralcloud.com'], error: 'array_atLeastOneValueFailed' },
+  ]
 
     runValidationTests(baseTests, 'array', { equalityCheck: 'eql' })
   }

package/test/tests/string.js

@@ -18,6 +18,8 @@ module.exports = {
       { name: 'Valid string from randomValue function', type: 'string', value: randomValue, expected: randomValue },
       { name: 'Invalid - uppercase vs lowercase', type: 'string', value: 'ABC', enum: ['abc'], error: 'string_notAnAllowedValue' },
       { name: 'Valid - uppercase vs lowercase', type: 'string', ignoreCase: true, value: 'ABC', enum: ['abc'], expected: 'ABC' },
+      { name: 'Invalid - value too short', type: 'string', minLength: 5, value: 'ABC', error: 'string_stringTooShort_minLength5' },
+      { name: 'Valid - value is longer than 5 chars', type: 'string', minLength: 5, value: 'ABCDEF', expected: 'ABCDEF' },
     ]
 
     runValidationTests(baseTests, 'string', { equalityCheck: 'eql' })

package/test/tests/url.js

@@ -7,7 +7,7 @@ module.exports = {
     const baseTests = [
       { name: 'Valid url', type: 'url', value: 'https://www.admiralcloud.com', expected: 'https://www.admiralcloud.com' },
       { name: 'Invalid url', type: 'url', value: 'www.admiralcloud.com', error: 'url_notAValidURL' },
-      { name: 'Invalid url - wrong protocol', type: 'url', value: 'ftp://www.admiralcloud.com', error: 'url_notAValidURL' },
+      { name: 'Invalid url - wrong protocol', type: 'url', value: 'ftp://www.admiralcloud.com', error: 'url_notAValidURL' }
     ]
 
     runValidationTests(baseTests, 'url')