fix: add timeout for updating state

Signed-off-by: Alan D. Tse <alandtse@gmail.com>
spdx · Oct 5, 2019 · 853e4bf · 853e4bf
1 parent 4887441
commit 853e4bf
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/app/scripts/background.js b/app/scripts/background.js
@@ -139,12 +139,21 @@ function handleMessage(request, sender, sendResponse) {
           };
         } else comparequeue.push({ selection: selection, tabId: activeTabId });
         status[activeTabId] = "Pending";
-        if (updating) {
+        var timeElapsed = updating ? Date.now() - updating : 0;
+        if (updating && timeElapsed <= 120000) {
           console.log(
-            "Update pending; queing compare for tab %s; %s queued",
+            "Update pending %s seconds; queing compare for tab %s; %s queued",
+            (timeElapsed / 1000).toFixed(2),
             activeTabId,
             comparequeue.length
           );
+        } else if (updating) {
+          console.log(
+            "Update pending %s seconds exceeded timeout; forcing load list; %s queued",
+            (timeElapsed / 1000).toFixed(2),
+            comparequeue.length
+          );
+          loadList();
         } else {
           console.log(
             "License load needed; queing compare for tab %s; %s queued",
@@ -591,7 +600,7 @@ function updateList() {
   if (updating) {
     console.log("Ignoring redundant update request");
   } else {
-    updating = true;
+    updating = Date.now();
     licensesLoaded = 0;
     dowork({ command: "updatelicenselist" });
   }