Merge pull request #276 from MindscapeHQ/fix-null-comparisons-when-un…

…defined Fix null comparisons when undefined
MindscapeHQ · Apr 26, 2018 · 171ea5d · 171ea5d
2 parents 8ac95a0 + e747e0b
commit 171ea5d
Show file tree

Hide file tree

Showing 13 changed files with 36 additions and 28 deletions.
diff --git a/bower.json b/bower.json
@@ -1,6 +1,6 @@
 {
   "name": "raygun4js",
-  "version": "2.9.2",
+  "version": "2.9.3",
   "homepage": "http://raygun.io",
   "authors": [
     "Mindscape <[email protected]>"

diff --git a/dist/raygun.js b/dist/raygun.js
@@ -1,4 +1,4 @@
-/*! Raygun4js - v2.9.2 - 2018-04-19
+/*! Raygun4js - v2.9.3 - 2018-04-27
 * https://github.com/MindscapeHQ/raygun4js
 * Copyright (c) 2018 MindscapeHQ; Licensed MIT */
 (function(window, undefined) {
@@ -2693,7 +2693,9 @@ var raygunFactory = function (window, $, undefined) {
         if (_filteredKeys) {
             for (var i = 0; i < _filteredKeys.length; i++) {
                 if (typeof _filteredKeys[i] === 'object' && typeof _filteredKeys[i].exec === 'function') {
-                    if (_filteredKeys[i].exec(key) !== null) {
+                    var executedFilter = _filteredKeys[i].exec(key);
+
+                    if (executedFilter !== null && executedFilter !== undefined) {
                         return '[removed by filter]';
                     }
                 }
@@ -2806,7 +2808,7 @@ var raygunFactory = function (window, $, undefined) {
             }
 
 
-            if (stackTrace.stack[0].url !== null && stackTrace.stack[0].url.indexOf(domain) === -1) {
+            if (stackTrace.stack[0].url !== null && stackTrace.stack[0] !== undefined && stackTrace.stack[0].url.indexOf(domain) === -1) {
                 var allowedDomainFound = false;
 
                 for (var i in _whitelistedScriptDomains) {
@@ -2982,7 +2984,7 @@ var raygunFactory = function (window, $, undefined) {
                 },
                 'Client': {
                     'Name': 'raygun-js',
-                    'Version': '2.9.2'
+                    'Version': '2.9.3'
                 },
                 'UserCustomData': finalCustomData,
                 'Tags': options.tags,

diff --git a/dist/raygun.min.js b/dist/raygun.min.js
diff --git a/dist/raygun.min.js.map b/dist/raygun.min.js.map
diff --git a/dist/raygun.umd.js b/dist/raygun.umd.js
@@ -1,4 +1,4 @@
-/*! Raygun4js - v2.9.2 - 2018-04-19
+/*! Raygun4js - v2.9.3 - 2018-04-27
 * https://github.com/MindscapeHQ/raygun4js
 * Copyright (c) 2018 MindscapeHQ; Licensed MIT */
 // https://github.com/umdjs/umd/blob/master/templates/returnExportsGlobal.js
@@ -2742,7 +2742,9 @@ var raygunFactory = function (window, $, undefined) {
         if (_filteredKeys) {
             for (var i = 0; i < _filteredKeys.length; i++) {
                 if (typeof _filteredKeys[i] === 'object' && typeof _filteredKeys[i].exec === 'function') {
-                    if (_filteredKeys[i].exec(key) !== null) {
+                    var executedFilter = _filteredKeys[i].exec(key);
+
+                    if (executedFilter !== null && executedFilter !== undefined) {
                         return '[removed by filter]';
                     }
                 }
@@ -2855,7 +2857,7 @@ var raygunFactory = function (window, $, undefined) {
             }
 
 
-            if (stackTrace.stack[0].url !== null && stackTrace.stack[0].url.indexOf(domain) === -1) {
+            if (stackTrace.stack[0].url !== null && stackTrace.stack[0] !== undefined && stackTrace.stack[0].url.indexOf(domain) === -1) {
                 var allowedDomainFound = false;
 
                 for (var i in _whitelistedScriptDomains) {
@@ -3031,7 +3033,7 @@ var raygunFactory = function (window, $, undefined) {
                 },
                 'Client': {
                     'Name': 'raygun-js',
-                    'Version': '2.9.2'
+                    'Version': '2.9.3'
                 },
                 'UserCustomData': finalCustomData,
                 'Tags': options.tags,

diff --git a/dist/raygun.umd.min.js b/dist/raygun.umd.min.js
diff --git a/dist/raygun.umd.min.js.map b/dist/raygun.umd.min.js.map
diff --git a/dist/raygun.vanilla.js b/dist/raygun.vanilla.js
@@ -1,4 +1,4 @@
-/*! Raygun4js - v2.9.2 - 2018-04-19
+/*! Raygun4js - v2.9.3 - 2018-04-27
 * https://github.com/MindscapeHQ/raygun4js
 * Copyright (c) 2018 MindscapeHQ; Licensed MIT */
 (function(window, undefined) {
@@ -2625,7 +2625,9 @@ var raygunFactory = function (window, $, undefined) {
         if (_filteredKeys) {
             for (var i = 0; i < _filteredKeys.length; i++) {
                 if (typeof _filteredKeys[i] === 'object' && typeof _filteredKeys[i].exec === 'function') {
-                    if (_filteredKeys[i].exec(key) !== null) {
+                    var executedFilter = _filteredKeys[i].exec(key);
+
+                    if (executedFilter !== null && executedFilter !== undefined) {
                         return '[removed by filter]';
                     }
                 }
@@ -2738,7 +2740,7 @@ var raygunFactory = function (window, $, undefined) {
             }
 
 
-            if (stackTrace.stack[0].url !== null && stackTrace.stack[0].url.indexOf(domain) === -1) {
+            if (stackTrace.stack[0].url !== null && stackTrace.stack[0] !== undefined && stackTrace.stack[0].url.indexOf(domain) === -1) {
                 var allowedDomainFound = false;
 
                 for (var i in _whitelistedScriptDomains) {
@@ -2914,7 +2916,7 @@ var raygunFactory = function (window, $, undefined) {
                 },
                 'Client': {
                     'Name': 'raygun-js',
-                    'Version': '2.9.2'
+                    'Version': '2.9.3'
                 },
                 'UserCustomData': finalCustomData,
                 'Tags': options.tags,

diff --git a/dist/raygun.vanilla.min.js b/dist/raygun.vanilla.min.js
diff --git a/dist/raygun.vanilla.min.js.map b/dist/raygun.vanilla.min.js.map
diff --git a/package.json b/package.json
@@ -3,7 +3,7 @@
   "main": "dist/raygun.umd.js",
   "title": "Raygun4js",
   "description": "Raygun.io plugin for JavaScript",
-  "version": "2.9.2",
+  "version": "2.9.3",
   "homepage": "https://github.com/MindscapeHQ/raygun4js",
   "author": {
     "name": "MindscapeHQ",

diff --git a/raygun4js.nuspec b/raygun4js.nuspec
@@ -2,7 +2,7 @@
 <package xmlns="http://schemas.microsoft.com/packaging/2010/07/nuspec.xsd">
   <metadata>
     <id>raygun4js</id>
-    <version>2.9.2</version>
+    <version>2.9.3</version>
     <title>Raygun4js</title>
     <authors>Mindscape Limited</authors>
     <owners>Mindscape Limited</owners>

diff --git a/src/raygun.js b/src/raygun.js
@@ -495,7 +495,9 @@ var raygunFactory = function (window, $, undefined) {
         if (_filteredKeys) {
             for (var i = 0; i < _filteredKeys.length; i++) {
                 if (typeof _filteredKeys[i] === 'object' && typeof _filteredKeys[i].exec === 'function') {
-                    if (_filteredKeys[i].exec(key) !== null) {
+                    var executedFilter = _filteredKeys[i].exec(key);
+
+                    if (executedFilter !== null && executedFilter !== undefined) {
                         return '[removed by filter]';
                     }
                 }
@@ -608,7 +610,7 @@ var raygunFactory = function (window, $, undefined) {
             }
 
 
-            if (stackTrace.stack[0].url !== null && stackTrace.stack[0].url.indexOf(domain) === -1) {
+            if (stackTrace.stack[0].url !== null && stackTrace.stack[0] !== undefined && stackTrace.stack[0].url.indexOf(domain) === -1) {
                 var allowedDomainFound = false;
 
                 for (var i in _whitelistedScriptDomains) {