ac-sanitizer 4.0.1 → 4.0.3

package/.github/workflows/node.js.yml

@@ -0,0 +1,30 @@
+# This workflow will do a clean installation of node dependencies, cache/restore them, build the source code and run tests across different versions of node
+# For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions
+
+name: Node.js CI
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        node-version: [16.x, 18.x]
+        # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
+
+    steps:
+    - uses: actions/checkout@v3
+    - name: Use Node.js ${{ matrix.node-version }}
+      uses: actions/setup-node@v3
+      with:
+        node-version: ${{ matrix.node-version }}
+
+    - run: yarn install
+    - run: yarn run test

package/CHANGELOG.md

@@ -1,3 +1,38 @@
+<a name="4.0.3"></a>
+
+## [4.0.3](https://github.com/mmpro/ac-sanitizer/compare/v4.0.2..v4.0.3) (2023-07-21 19:16:42)
+
+
+### Bug Fix
+
+* **App:** Package updated | MP | [32ba9d92894d54a4575a0765bafbe5f1be6f8edf](https://github.com/mmpro/ac-sanitizer/commit/32ba9d92894d54a4575a0765bafbe5f1be6f8edf)    
+Packages updated  
+Related issues: [/issues#undefined](https://github.com//issues/undefined)
+### Tests
+
+* **App:** Fixed test | MP | [05e575d49d3254bc6191a1b3e950bb7f89e07fd0](https://github.com/mmpro/ac-sanitizer/commit/05e575d49d3254bc6191a1b3e950bb7f89e07fd0)    
+Fixed nested object test  
+Related issues: [/issues#undefined](https://github.com//issues/undefined)
+<a name="4.0.2"></a>
+
+## [4.0.2](https://github.com/mmpro/ac-sanitizer/compare/v4.0.1..v4.0.2) (2023-06-18 11:35:24)
+
+
+### Bug Fix
+
+* **App:** If base64 is an object, parse the converted string | MP | [cf9a49d9df121c7829932ae37192fd22f3d98438](https://github.com/mmpro/ac-sanitizer/commit/cf9a49d9df121c7829932ae37192fd22f3d98438)    
+Base64 might be a string or a stringified object. Try parsing the string to determine if it is an object  
+Related issues: [undefined/undefined#master](undefined/browse/master)
+### Chores
+
+* **App:** Updated packages | MP | [49da6b224e9382aa7cc6a298b2a0af06648931b8](https://github.com/mmpro/ac-sanitizer/commit/49da6b224e9382aa7cc6a298b2a0af06648931b8)    
+Updated packages  
+Related issues: [undefined/undefined#master](undefined/browse/master)
+### Chores
+
+* **App:** Add Github workflow | MP | [72668a139dda85330516077ef471af4aa4b68c3b](https://github.com/mmpro/ac-sanitizer/commit/72668a139dda85330516077ef471af4aa4b68c3b)    
+Add Github workflow  
+Related issues: [undefined/undefined#master](undefined/browse/master)
 <a name="4.0.1"></a>
 
 ## [4.0.1](https://github.com/mmpro/ac-sanitizer/compare/v4.0.0..v4.0.1) (2023-04-18 18:38:15)

package/README.md

@@ -1,6 +1,8 @@
 # AC Sanitizer
 Sanitizes  payloads with given field definitions
 
+[![Node.js CI](https://github.com/AdmiralCloud/ac-sanitizer/actions/workflows/node.js.yml/badge.svg)](https://github.com/AdmiralCloud/ac-sanitizer/actions/workflows/node.js.yml)
+
 ### Version 4 - Breaking changes
 Version 4 requires Node 16.
 

package/index.js

@@ -330,6 +330,13 @@ const sanitizer = function() {
           if (!validator.isBase64(_.padEnd(value, (l+pad), '='))) error = { message: fieldName + '_notABase64String' }
           else if (field.convert) {
             _.set(paramsToCheck, fieldName, Buffer.from(value, 'base64').toString())
+            // the value might be a stringified object - try converting it
+            try {
+              _.set(paramsToCheck, fieldName, JSON.parse(_.get(paramsToCheck, fieldName)))
+            } 
+            catch(e) {
+              // ignore
+            }
           }
         }
       }

package/package.json

@@ -4,27 +4,27 @@
   "author": "Mark Poepping (https://www.admiralcloud.com)",
   "license": "MIT",
   "repository": "admiralcloud/ac-sanitizer",
-  "version": "4.0.1",
+  "version": "4.0.3",
   "homepage": "https://www.admiralcloud.com",
   "dependencies": {
     "ac-countrylist": "^1.0.7",
     "ac-file-extensions": "^2.0.5",
     "ac-ip": "^3.0.1",
     "chai": "^4.3.7",
-    "date-and-time": "^2.4.3",
-    "hashids": "^2.2.11",
+    "date-and-time": "^3.0.2",
+    "hashids": "^2.3.0",
     "lodash": "^4.17.21",
     "validator": "^13.9.0"
   },
   "devDependencies": {
-    "ac-semantic-release": "^0.4.0",
-    "eslint": "^8.38.0",
-    "mocha": "^10.2.0",
-    "nyc": "^15.1.0"
+    "ac-semantic-release": "^0.4.2",
+    "c8": "^8.0.0",
+    "eslint": "^8.45.0",
+    "mocha": "^10.2.0"
   },
   "scripts": {
     "test": "mocha --reporter spec",
-    "coverage": "nyc yarn run test"
+    "coverage": "./node_modules/c8/bin/c8.js yarn test"
   },
   "engines": {
     "node": ">=16.0.0"

package/test/tests/base64.js

@@ -13,6 +13,8 @@ module.exports = {
       { name: 'Invalid base64 with convert', type: 'base64', convert: true, value: 123, error: 'base64_mustBeString' },
       { name: 'Base64 app.admiralcloud.com - requires padding', type: 'base64', value: 'aHR0cHM6Ly9hcHAuYWRtaXJhbGNsb3VkLmNvbQ', convert: true, expected: 'https://app.admiralcloud.com' },
       { name: 'Base64 app.admiralcloud.com - with padding', type: 'base64', value: 'aHR0cHM6Ly9hcHAuYWRtaXJhbGNsb3VkLmNvbQ==', convert: true, expected: 'https://app.admiralcloud.com' },
+      { name: 'Base64 encoded object', type: 'base64', value: 'eyJ1c2VySWQiOjEyMywiY3VzdG9tZXJJZCI6MTQ2LCJyZWFzb24iOiJCZWNhdXNlIEkgY2FuIn0=', convert: true, expected: { userId: 123, customerId: 146, reason: 'Because I can' } },
+      { name: 'Base64 encoded object', type: 'base64', value: 'eyJ1c2VySWQiOjEyMywiY3VzdG9tZXJJZCI6MTQ2LCJyZWFzbOiJCZWNhdXNlIEkgY2FuIn0=', convert: true, error: 'base64_notABase64String' },
     ]
 
     _.forEach(baseTests, (test) => {
@@ -34,7 +36,7 @@ module.exports = {
           }
         }
         else {
-          expect(_.get(r, 'params.base64')).to.equal(_.get(test, 'expected'))
+          expect(_.get(r, 'params.base64')).to.eql(_.get(test, 'expected'))
         }
         return done()
       })

package/test/tests/object.js

@@ -199,6 +199,29 @@ module.exports = {
           }
         },
       },
+      {
+        name: "Object with complex nested properties",
+        type: "object",
+        properties: [
+          { field: "boo", type: "boolean" },
+          { field: "nested", type: "object", properties: [
+            { field: 'setId', type: 'boolean', required: false }
+          ] }
+        ],
+        value: {
+          boo: true,
+          enum: "blue",
+          nested: {
+            setId: false
+          }
+        },
+        expected: {
+          boo: true,
+          nested: {
+            setId: false
+          }
+        },
+      },
     ];
 
     _.forEach(baseTests, (test) => {