Bugfix: verify ip before net.contains

2021-04-06 08:56:33 -04:00 · 2021-04-06 08:56:33 -04:00 · 6bd7492f74
commit 6bd7492f74
parent a8046e2c2b
1 changed files with 3 additions and 1 deletions
--- a/server/src/lib/paloalto.js
+++ b/server/src/lib/paloalto.js
@ -69,7 +69,9 @@ async function updateUserIdMappings() {

    const net = ip.cidrSubnet(pAHost.cidr)

-    const devices = allDevices.filter(device => net.contains(device.ip))
+    const devices = allDevices.filter(
+      ({ ip }) => isIPv4(ip) && net.contains(ip)
+    )

    try {
      // console.log(pAHost.id, pAHost.description, devices.length)