Merge pull request #6 from duckduckgo/sam/validator-fixes

Live pixel validation: remove cache-buster before parameter validation

Author: sammacbeth

bin/validate_live_pixel.mjs

@@ -1,3 +1,5 @@
+#!/usr/bin/env node
+
 import fs from 'fs';
 import JSON5 from 'json5';
 

src/params_validator.mjs

@@ -140,7 +140,8 @@ export class ParamsValidator {
 
         const urlSplit = url.split('/')[2].split('?');
         const livePixelName = urlSplit[0].replaceAll('_', '.');
-        const livePixelRequestParams = urlSplit[1];
+        // grab pixel parameters with any preciding cache buster removed
+        const livePixelRequestParams = /^([0-9]+&)?(.*)$/.exec(urlSplit[1] || '')[2];
 
         // 1) Validate pixel name if it's parameterized
         if (livePixelName.length > prefix.length) {

tests/live_pixel_validation_test.mjs

@@ -46,6 +46,13 @@ describe('No common params nor suffixes', () => {
         const expectedErrors = ["must NOT have additional properties. Found extra property 'param2'"];
         expect(errors).to.have.members(expectedErrors);
     });
+
+    it('ignores cache buster', () => {
+        const prefix = 'simplePixel';
+        const url = `/t/${prefix}?12345&param1=true`;
+        const errors = paramsValidator.validateLivePixels(pixelDefs[prefix], prefix, url);
+        expect(errors).to.be.empty;
+    });
 });
 
 describe('Common params', () => {