From c74ef127d1d5644ee1b04da7315cb06ad5f08f69 Mon Sep 17 00:00:00 2001
From: Jokob-sk <jokob.sk@gmail.com>
Date: Fri, 5 May 2023 19:08:10 +1000
Subject: [PATCH] update only unknown vendors

---
 back/pialert.py | 43 ++++++++++++++++++++++++++++++++++++-------
 docs/ICONS.md   |  2 +-
 2 files changed, 37 insertions(+), 8 deletions(-)

diff --git a/back/pialert.py b/back/pialert.py
index f3ddec04..86eb1f1d 100755
--- a/back/pialert.py
+++ b/back/pialert.py
@@ -41,6 +41,7 @@ from cron_converter import Cron
 from pytz import timezone
 from json2table import convert
 import hashlib
+import multiprocessing
 
 #===============================================================================
 # SQL queries
@@ -124,12 +125,13 @@ def mylog(requestedDebugLevel, n):
 #-------------------------------------------------------------------------------
 def file_print (*args):
 
-    result = ''
-    
-    file = open(logPath + "/pialert.log", "a")    
+    result = ''    
+       
     for arg in args:                
         result += str(arg)
     print(result)
+
+    file = open(logPath + "/pialert.log", "a") 
     file.write(result + '\n')
     file.close()
 
@@ -655,7 +657,31 @@ def main ():
             if last_network_scan + datetime.timedelta(minutes=SCAN_CYCLE_MINUTES) < time_started:
                 last_network_scan = time_started
                 cycle = 1 # network scan
-                scan_network() 
+                # scan_network() 
+
+                #  DEBUG start ++++++++++++++++++++++++++++++++++++++++++++++++++++++
+                # Start scan_network as a process                
+
+                p = multiprocessing.Process(target=scan_network)
+                p.start()
+
+                # Wait for 3600 seconds (max 1h) or until process finishes
+                p.join(3600)
+
+                # If thread is still active
+                if p.is_alive():
+                    print "DEBUG scan_network running too long - let's kill it"
+                    mylog('info', ['    DEBUG scan_network running too long - let\'s kill it'])
+
+                    # Terminate - may not work if process is stuck for good
+                    p.terminate()
+                    # OR Kill - will work for sure, no chance for process to finish nicely however
+                    # p.kill()
+
+                    p.join()
+
+                #  DEBUG end ++++++++++++++++++++++++++++++++++++++++++++++++++++++
+                
             
             # Reporting   
             if cycle in check_report:
@@ -951,7 +977,10 @@ def update_devices_MAC_vendors (pArg = ''):
 
     # All devices loop
     mylog('verbose', ['    Searching devices vendor'])    
-    for device in sql.execute ("SELECT * FROM Devices") :
+    for device in sql.execute ("""SELECT * FROM Devices
+                                  WHERE dev_Vendor = '(unknown)' 
+                                     OR dev_Vendor =''
+                                     OR dev_Vendor IS NULL""") :
         # Search vendor in HW Vendors DB
         vendor = query_MAC_vendor (device['dev_MAC'])
         if vendor == -1 :
@@ -1081,9 +1110,9 @@ def scan_network ():
     update_devices_names()
 
     # Void false connection - disconnections
-    mylog('verbose', ['    Voiding false (ghost) disconnections'])
+    mylog('verbose', ['    Voiding false (ghost) disconnections'])    
     void_ghost_disconnections ()
-  
+
     # Pair session events (Connection / Disconnection)
     mylog('verbose', ['    Pairing session events (connection / disconnection) '])
     pair_sessions_events()  
diff --git a/docs/ICONS.md b/docs/ICONS.md
index aa126873..67b4cc0d 100755
--- a/docs/ICONS.md
+++ b/docs/ICONS.md
@@ -15,6 +15,6 @@ You can assign icons individually on each device in the Details tab.
   1. For any value that is only prefixed with `fa-`, you can enter the value directly, such as `server`, `tv`, `ethernet`. 
   2. If you want to add another classname, e.g. `fa-brands`, you can enter `brands fa-[fontawesome-icon-name]`, so for `apple` that is using the syntax`fa-brands fa-apple`, you would enter `brands fa-apple`.
 
-- If you want to mass-apply an icon to all devices of the same device type (Field marked (4) in the above screenshot), you can click the copy button (Marked (1) in the above screenshot). A confirmation prompt is dispalyed. If you proceed, icons of all devices set to the same device type as the current device, will be overwritten with the current devices icon.
+- If you want to mass-apply an icon to all devices of the same device type (Field marked (4) in the above screenshot), you can click the copy button (Marked (1) in the above screenshot). A confirmation prompt is displayed. If you proceed, icons of all devices set to the same device type as the current device, will be overwritten with the current device's icon.
 
 - The dropdown (3) contains all icons already used in the app for device icons. You need to navigate away or refresh the page once you add a new icon.