From eb58fb681f5a1d0d29b28bab85853bb7545937df Mon Sep 17 00:00:00 2001
From: Zachary Keeping <zacharykeeping@icloud.com>
Date: Wed, 29 Nov 2023 17:26:33 +1100
Subject: [PATCH 1/3] Fix whitelisted URLs count and update ignore URL binding

---
 api/functions/index.js                               |  2 +-
 docker/utils.js                                      | 12 +++++-------
 .../components/misccomponents/UpdateIgnoreUrl.svelte |  2 +-
 3 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/api/functions/index.js b/api/functions/index.js
index e1be6ca8..b63cb770 100644
--- a/api/functions/index.js
+++ b/api/functions/index.js
@@ -279,7 +279,7 @@ app.post('/scanresult/:api/:buildId', async (req, res) => {
 		...lhrSummary,
 		...atrSummary,
 		totalScanned,
-		whiteListed,
+		totalWhitelisted: whiteListed.length,
 		scanDuration,
 		url,
 		cloc,
diff --git a/docker/utils.js b/docker/utils.js
index 4e290d61..a62cfdc2 100644
--- a/docker/utils.js
+++ b/docker/utils.js
@@ -436,23 +436,21 @@ exports.processBrokenLinks = (
         whitelistedUrls
           .filter(
             (ig) =>
-              ig.ignoreOn === ignoreOn &&
+              (ig.ignoreOn === "all" || ig.ignoreOn === ignoreOn) &&
               (+ig.ignoreDuration === -1 ||
                 diffInDaysToNow(new Date(ig.effectiveFrom)) <
                   +ig.ignoreDuration)
           )
           .map((ig) => ig.urlToIgnore)
-          .filter((ignorePattern) => minimatch(url, ignorePattern)).length > 0
+          .filter((ignorePattern) => 
+            minimatch(url.src, ignorePattern) || minimatch(url.dst, ignorePattern)
+          ).length > 0
       );
     };
 
     // return the URL only
     const all = badUrls
-      .filter(
-        (url) =>
-          isInIgnoredList(url.dst, "all", whitelistedUrls) ||
-          isInIgnoredList(url.dst, startUrl, whitelistedUrls)
-      )
+      .filter((url) => isInIgnoredList(url, startUrl))
       .map((x) => x.dst);
     return [...new Set(all)];
   };
diff --git a/ui/src/components/misccomponents/UpdateIgnoreUrl.svelte b/ui/src/components/misccomponents/UpdateIgnoreUrl.svelte
index 35250aba..b75a735e 100644
--- a/ui/src/components/misccomponents/UpdateIgnoreUrl.svelte
+++ b/ui/src/components/misccomponents/UpdateIgnoreUrl.svelte
@@ -152,7 +152,7 @@
               class="hidden"
               id="radio2"
               bind:group={ignoreOn}
-              value={url}
+              value={scanUrl}
               disabled={editing}
             />
             <label for="radio2" class:disabled={editing} class="flex items-center">

From cc3ee6b818260210552a56b573fd3d846a97a5fd Mon Sep 17 00:00:00 2001
From: Zachary Keeping <zacharykeeping@icloud.com>
Date: Thu, 30 Nov 2023 09:03:36 +1100
Subject: [PATCH 2/3] Update frontend display

---
 ui/src/components/linkauditorcomponents/DetailsByDest.svelte   | 2 +-
 ui/src/components/linkauditorcomponents/DetailsByReason.svelte | 2 +-
 ui/src/components/linkauditorcomponents/DetailsBySource.svelte | 2 +-
 ui/src/utils/utils.js                                          | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/ui/src/components/linkauditorcomponents/DetailsByDest.svelte b/ui/src/components/linkauditorcomponents/DetailsByDest.svelte
index 0f20870f..9de97211 100644
--- a/ui/src/components/linkauditorcomponents/DetailsByDest.svelte
+++ b/ui/src/components/linkauditorcomponents/DetailsByDest.svelte
@@ -25,7 +25,7 @@
     destinationsKeys = Object.keys(destinations);
 
     ignoredChecks = builds.reduce((acc, val) => {
-      acc[val.dst] = isInIgnored(val.dst, $ignoredUrls$);
+      acc[val.dst] = isInIgnored(val, $ignoredUrls$);
       return acc;
     }, {});
   }
diff --git a/ui/src/components/linkauditorcomponents/DetailsByReason.svelte b/ui/src/components/linkauditorcomponents/DetailsByReason.svelte
index a4bd91db..abdf94f2 100644
--- a/ui/src/components/linkauditorcomponents/DetailsByReason.svelte
+++ b/ui/src/components/linkauditorcomponents/DetailsByReason.svelte
@@ -25,7 +25,7 @@
     reasonsKeys = Object.keys(reasons);
 
     ignoredChecks = builds.reduce((acc, val) => {
-      acc[val.dst] = isInIgnored(val.dst, $ignoredUrls$);
+      acc[val.dst] = isInIgnored(val, $ignoredUrls$);
       return acc;
     }, {});
   }
diff --git a/ui/src/components/linkauditorcomponents/DetailsBySource.svelte b/ui/src/components/linkauditorcomponents/DetailsBySource.svelte
index ef1d0715..345d042a 100644
--- a/ui/src/components/linkauditorcomponents/DetailsBySource.svelte
+++ b/ui/src/components/linkauditorcomponents/DetailsBySource.svelte
@@ -23,7 +23,7 @@
     sources = groupBy(props(["src"]))(builds);
     sourcesKeys = Object.keys(sources);
     ignoredChecks = builds.reduce((acc, val) => {
-      acc[val.dst] = isInIgnored(val.dst, $ignoredUrls$);
+      acc[val.dst] = isInIgnored(val, $ignoredUrls$);
       return acc;
     }, {});
   }
diff --git a/ui/src/utils/utils.js b/ui/src/utils/utils.js
index c3321ffa..21da30dd 100644
--- a/ui/src/utils/utils.js
+++ b/ui/src/utils/utils.js
@@ -134,7 +134,7 @@ export const getMatchingIgnoredRules = (url, list) => {
   const date = new Date();
   return list.filter((item) => {
     const pattern = item.urlToIgnore;
-    if (globMatchUrl(pattern, url)) {
+    if (globMatchUrl(pattern, url.src) || globMatchUrl(pattern, url.dst)) {
       const effectiveFrom = new Date(item.effectiveFrom);
       const timeElapsed = (date - effectiveFrom) / 86400000;
       return (item.ignoreDuration > 0 && timeElapsed < item.ignoreDuration) || item.ignoreDuration === -1;

From 0181a14e73f487eb4792d10d9de0692b5f3e4236 Mon Sep 17 00:00:00 2001
From: Zachary Keeping <zacharykeeping@icloud.com>
Date: Thu, 30 Nov 2023 09:53:43 +1100
Subject: [PATCH 3/3] Filter on scanUrl

---
 .../components/linkauditorcomponents/DetailsByDest.svelte | 5 +++--
 .../linkauditorcomponents/DetailsByReason.svelte          | 6 ++++--
 .../linkauditorcomponents/DetailsBySource.svelte          | 6 ++++--
 .../components/linkauditorcomponents/DetailsTable.svelte  | 7 ++++---
 ui/src/containers/LinkReport.svelte                       | 3 ++-
 ui/src/utils/utils.js                                     | 8 ++++----
 6 files changed, 21 insertions(+), 14 deletions(-)

diff --git a/ui/src/components/linkauditorcomponents/DetailsByDest.svelte b/ui/src/components/linkauditorcomponents/DetailsByDest.svelte
index 9de97211..80c92320 100644
--- a/ui/src/components/linkauditorcomponents/DetailsByDest.svelte
+++ b/ui/src/components/linkauditorcomponents/DetailsByDest.svelte
@@ -11,6 +11,7 @@
   const dispatch = createEventDispatcher();
 
   export let builds = [];
+  export let scanUrl;
 
   const ignore = url => dispatch("ignore", url);
   let destinations;
@@ -25,7 +26,7 @@
     destinationsKeys = Object.keys(destinations);
 
     ignoredChecks = builds.reduce((acc, val) => {
-      acc[val.dst] = isInIgnored(val, $ignoredUrls$);
+      acc[val.dst] = isInIgnored(val, scanUrl, $ignoredUrls$);
       return acc;
     }, {});
   }
@@ -47,7 +48,7 @@
   const deleteIgnore = async (url) => {
     deleteUrl = url;
     loadingChecks[url] = true;
-    const rules = getMatchingIgnoredRules(url, $ignoredUrls$);
+    const rules = getMatchingIgnoredRules(url, scanUrl, $ignoredUrls$);
     try {
       for await (const rule of rules) {
         await deleteIgnoreUrl(rule, $userSession$);
diff --git a/ui/src/components/linkauditorcomponents/DetailsByReason.svelte b/ui/src/components/linkauditorcomponents/DetailsByReason.svelte
index abdf94f2..8371a684 100644
--- a/ui/src/components/linkauditorcomponents/DetailsByReason.svelte
+++ b/ui/src/components/linkauditorcomponents/DetailsByReason.svelte
@@ -12,6 +12,8 @@
   const dispatch = createEventDispatcher();
   const ignore = url => dispatch("ignore", url);
 
+  export let scanUrl;
+
   let reasons;
   let reasonsKeys = [];
   let hiddenRows = {};
@@ -25,7 +27,7 @@
     reasonsKeys = Object.keys(reasons);
 
     ignoredChecks = builds.reduce((acc, val) => {
-      acc[val.dst] = isInIgnored(val, $ignoredUrls$);
+      acc[val.dst] = isInIgnored(val, scanUrl, $ignoredUrls$);
       return acc;
     }, {});
   }
@@ -46,7 +48,7 @@
   const deleteIgnore = async (url) => {
     deleteUrl = url;
     loadingChecks[url] = true;
-    const rules = getMatchingIgnoredRules(url, $ignoredUrls$);
+    const rules = getMatchingIgnoredRules(url, scanUrl, $ignoredUrls$);
     try {
       for await (const rule of rules) {
         await deleteIgnoreUrl(rule, $userSession$);
diff --git a/ui/src/components/linkauditorcomponents/DetailsBySource.svelte b/ui/src/components/linkauditorcomponents/DetailsBySource.svelte
index 345d042a..be135621 100644
--- a/ui/src/components/linkauditorcomponents/DetailsBySource.svelte
+++ b/ui/src/components/linkauditorcomponents/DetailsBySource.svelte
@@ -12,6 +12,8 @@
   const dispatch = createEventDispatcher();
   const ignore = url => dispatch("ignore", url);
 
+  export let scanUrl;
+
   let sources;
   let sourcesKeys = [];
   let ignoredChecks = {};
@@ -23,7 +25,7 @@
     sources = groupBy(props(["src"]))(builds);
     sourcesKeys = Object.keys(sources);
     ignoredChecks = builds.reduce((acc, val) => {
-      acc[val.dst] = isInIgnored(val, $ignoredUrls$);
+      acc[val.dst] = isInIgnored(val, scanUrl, $ignoredUrls$);
       return acc;
     }, {});
   }
@@ -45,7 +47,7 @@
   const deleteIgnore = async (url) => {
     deleteUrl = url;
     loadingChecks[url] = true;
-    const rules = getMatchingIgnoredRules(url, $ignoredUrls$);
+    const rules = getMatchingIgnoredRules(url, scanUrl, $ignoredUrls$);
     try {
       for await (const rule of rules) {
         await deleteIgnoreUrl(rule, $userSession$);
diff --git a/ui/src/components/linkauditorcomponents/DetailsTable.svelte b/ui/src/components/linkauditorcomponents/DetailsTable.svelte
index 81b1eb28..f010214f 100644
--- a/ui/src/components/linkauditorcomponents/DetailsTable.svelte
+++ b/ui/src/components/linkauditorcomponents/DetailsTable.svelte
@@ -12,6 +12,7 @@
   export let builds = [];
   export let currentRoute;
   export let unscannableLinks;
+  export let scanUrl;
   
   let foundUnscannableLinks = [];
   foundUnscannableLinks = builds.filter(build => unscannableLinks.some(link => build.dst.includes(link)));
@@ -167,10 +168,10 @@
 
 {#if builds.length}
   {#if displayMode === 0}
-    <DetailsBySource {builds} on:ignore />
+    <DetailsBySource {builds} {scanUrl} on:ignore />
   {:else if displayMode === 1}
-    <DetailsByDest {builds} on:ignore />
+    <DetailsByDest {builds} {scanUrl} on:ignore />
   {:else}
-    <DetailsByReason {builds} on:ignore />
+    <DetailsByReason {builds} {scanUrl} on:ignore />
   {/if}
 {/if}
diff --git a/ui/src/containers/LinkReport.svelte b/ui/src/containers/LinkReport.svelte
index 38c25f31..aca9a22d 100644
--- a/ui/src/containers/LinkReport.svelte
+++ b/ui/src/containers/LinkReport.svelte
@@ -85,7 +85,8 @@
         on:ignore={url => showIgnore(data.buildDetails.summary.url, url, $userSession$)}
         builds={data.buildDetails ? data.buildDetails.brokenLinks : []}
         {currentRoute}
-        unscannableLinks={[]} 
+        unscannableLinks={[]}
+        scanUrl={data.buildDetails.summary.url}
       />
     </BuildDetailsSlot>
     {:catch error}
diff --git a/ui/src/utils/utils.js b/ui/src/utils/utils.js
index 21da30dd..346cedde 100644
--- a/ui/src/utils/utils.js
+++ b/ui/src/utils/utils.js
@@ -126,15 +126,15 @@ export const getLoadThresholdResult = (value) => ({
   errors: value.errors,
 });
 
-export const isInIgnored = (url, list) => {
-  return getMatchingIgnoredRules(url, list).length > 0;
+export const isInIgnored = (url, scanUrl, list) => {
+  return getMatchingIgnoredRules(url, scanUrl, list).length > 0;
 };
 
-export const getMatchingIgnoredRules = (url, list) => {
+export const getMatchingIgnoredRules = (url, scanUrl, list) => {
   const date = new Date();
   return list.filter((item) => {
     const pattern = item.urlToIgnore;
-    if (globMatchUrl(pattern, url.src) || globMatchUrl(pattern, url.dst)) {
+    if ((item.ignoreOn === "all" || item.ignoreOn === scanUrl) && (globMatchUrl(pattern, url.src) || globMatchUrl(pattern, url.dst))) {
       const effectiveFrom = new Date(item.effectiveFrom);
       const timeElapsed = (date - effectiveFrom) / 86400000;
       return (item.ignoreDuration > 0 && timeElapsed < item.ignoreDuration) || item.ignoreDuration === -1;