repalce for-in with 'normal' for loop in mqtt and http

onozuka · Nov 18, 2019 · 175a871 · 175a871
1 parent b4e2061
commit 175a871
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/node_modules/@node-red/nodes/core/network/10-mqtt.js b/packages/node_modules/@node-red/nodes/core/network/10-mqtt.js
@@ -129,7 +129,7 @@ module.exports = function(RED) {
                     // check if proxy is set in env
                     var noproxy;
                     if (noprox) {
-                        for (var i in noprox) {
+                        for (var i = 0; i < noprox.length; i += 1) {
                             if (this.brokerurl.indexOf(noprox[i].trim()) !== -1) { noproxy=true; }
                         }
                     }

diff --git a/packages/node_modules/@node-red/nodes/core/network/21-httprequest.js b/packages/node_modules/@node-red/nodes/core/network/21-httprequest.js
@@ -292,7 +292,7 @@ module.exports = function(RED) {
 
             var noproxy;
             if (noprox) {
-                for (var i in noprox) {
+                for (var i = 0; i < noprox.length; i += 1) {
                     if (url.indexOf(noprox[i]) !== -1) { noproxy=true; }
                 }
             }