ac-sanitizer 3.9.13 → 3.9.16

package/CHANGELOG.md

@@ -1,3 +1,39 @@
+<a name="3.9.16"></a>
+
+## [3.9.16](https://github.com/mmpro/ac-sanitizer/compare/v3.9.15..v3.9.16) (2022-04-24 14:23:01)
+
+
+### Bug Fix
+
+* **App:** Allow empty string for type integer | string | MP | [c06c64611a83f324d7736314e299d5fb768ab3a7](https://github.com/mmpro/ac-sanitizer/commit/c06c64611a83f324d7736314e299d5fb768ab3a7)    
+Allow empty string for type integer  
+string  
+Related issues: [/issues#undefined](https://github.com//issues/undefined)
+### Chores
+
+* **App:** Updated packages | MP | [f5cf0cabe083f991f455c47df596cbe27ba12054](https://github.com/mmpro/ac-sanitizer/commit/f5cf0cabe083f991f455c47df596cbe27ba12054)    
+Updated packages  
+Related issues: [undefined/undefined#master](undefined/browse/master)
+<a name="3.9.15"></a>
+
+## [3.9.15](https://github.com/mmpro/ac-sanitizer/compare/v3.9.14..v3.9.15) (2022-04-22 16:53:45)
+
+
+### Bug Fix
+
+* **App:** Remove entries from array which are not in enum | MP | [20fb76be489ff990e6777d0884ca32beea59b18e](https://github.com/mmpro/ac-sanitizer/commit/20fb76be489ff990e6777d0884ca32beea59b18e)    
+Remove entries from array which are not in enum  
+Related issues: [undefined/undefined#master](undefined/browse/master)
+<a name="3.9.14"></a>
+
+## [3.9.14](https://github.com/mmpro/ac-sanitizer/compare/v3.9.13..v3.9.14) (2022-04-20 11:41:13)
+
+
+### Bug Fix
+
+* **App:** Package updated | MP | [d683df4256efd105f1ced1681a3eaa6a06f0ca10](https://github.com/mmpro/ac-sanitizer/commit/d683df4256efd105f1ced1681a3eaa6a06f0ca10)    
+Packages updated  
+Related issues: [undefined/undefined#master](undefined/browse/master)
 <a name="3.9.13"></a>
 
 ## [3.9.13](https://github.com/mmpro/ac-sanitizer/compare/v3.9.12..v3.9.13) (2022-04-20 11:35:54)

package/index.js

@@ -96,7 +96,7 @@ const sanitizer = function() {
 
       /// SPECIAL FIELDS
       // special field - can be string or integer -> determine type and then use type settings
-      if (field.type === 'integer | string' && value) {
+      if (field.type === 'integer | string') {
         if (!_.isString(value) && !_.isFinite(parseInt(value))) {
           error = { message: fieldName + '_neitherStringNorInteger' }
           return {
@@ -104,7 +104,7 @@ const sanitizer = function() {
           }
         }
         if (_.isString(value)) {
-            field.type = 'string'
+          field.type = 'string'
         }
         else if (_.isFinite(parseInt(value))) {
           field.type = 'integer'
@@ -423,6 +423,9 @@ const sanitizer = function() {
           if (_.size(value) && !_.size(_.intersection(value, allowedValues))) {
             error = { message: fieldName + '_notAnAllowedValue', additionalInfo: { value } }
           }
+          // remove non-matching entries, but do not fail/return error
+          value = _.intersection(value, allowedValues)
+          _.set(paramsToCheck, fieldName, value)
         }
         else if (_.indexOf(allowedValues, value) < 0) {
           error = { message: fieldName + '_notAnAllowedValue', additionalInfo: { value } }

package/package.json

@@ -4,11 +4,11 @@
   "author": "Mark Poepping (https://www.admiralcloud.com)",
   "license": "MIT",
   "repository": "admiralcloud/ac-sanitizer",
-  "version": "3.9.13",
+  "version": "3.9.16",
   "homepage": "https://www.admiralcloud.com",
   "dependencies": {
     "ac-countrylist": "^1.0.6",
-    "ac-file-extensions": "^2.0.3",
+    "ac-file-extensions": "^2.0.5",
     "ac-ip": "^1.3.7",
     "date-and-time": "^2.3.1",
     "hashids": "^2.2.10",
@@ -17,7 +17,7 @@
   },
   "devDependencies": {
     "ac-semantic-release": "^0.3.0",
-    "eslint": "^8.13.0",
+    "eslint": "^8.14.0",
     "expect": "^27.5.1",
     "mocha": "^9.2.2",
     "nyc": "^15.1.0"

package/test/tests/array.js

@@ -12,6 +12,7 @@ module.exports = {
       { name: 'Invalid array', type: 'array', value: 'a', error: 'array_notAnArray' },
       { name: 'Array with enum match', type: 'array', value: ['video'], enum: ['audio', 'video'], expected: ['video'] },
       { name: 'Array without enum match', type: 'array', value: ['cookie'], enum: ['audio', 'video'], error: 'array_notAnAllowedValue' },
+      { name: 'Array with one non-matching - should filter out that value', type: 'array', value: ['video', 'cookie'], enum: ['audio', 'video'], expected: ['video'] },
       { name: 'Array with enum with placeholder countrylist', type: 'array', value: ['Laos'], enum: 'countrylist', expected: ['Laos'] },
       { name: 'Array with enum with placeholder countrylist - fail', type: 'array', value: ['Paris'], enum: 'countrylist', error: 'array_notAnAllowedValue' },
       { name: 'Array with enum with placeholder iso-639-2', type: 'array', value: ['deu'], enum: 'iso-639-2', expected: ['deu'] },

package/test/tests/stringOrInteger.js

@@ -8,6 +8,7 @@ module.exports = {
 
     const baseTests = [
       { name: 'Valid string', type: 'integer | string', value: 'abc-123', expected: 'abc-123' },
+      { name: 'Valid empty string', type: 'integer | string', value: '', expected: '', minLength: 0 },
       { name: 'Valid integer', type: 'integer | string', value: 123, expected:  123 },
       { name: 'Invalid - neither string nor integer', type: 'integer | string', value: ['abc'], error: 'stringOrInteger_neitherStringNorInteger' },
     ]