(whimsy) branch master updated: Try to fix scanning errors

[sebb]

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new 5b68b1cf Try to fix scanning errors
5b68b1cf is described below

commit 5b68b1cff7427e8de1199702b8d511a2a9fc172e
Author: Sebb <s...@apache.org>
AuthorDate: Thu Jan 4 11:11:38 2024 +0000

    Try to fix scanning errors
---
 tools/scan-page.js | 4 ++--
 tools/site-scan.rb | 1 -
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/tools/scan-page.js b/tools/scan-page.js
index 0748a283..6bb79e61 100755
--- a/tools/scan-page.js
+++ b/tools/scan-page.js
@@ -52,8 +52,8 @@ if (!isASFhost(inithost)) {
     }
   });
   let result = await page.goto(target);
-  let status = result._status;
-  if (status != 200) {
+  let status = result._status; // now seems to be null if it completed OK?
+  if (status && status != 200) {
     let url = result._url;
     let error = `Status ${status} for ${url}`;
     throw new Error(error);
diff --git a/tools/site-scan.rb b/tools/site-scan.rb
index c9b58718..e45a1aa0 100755
--- a/tools/site-scan.rb
+++ b/tools/site-scan.rb
@@ -272,7 +272,6 @@ podlings = {}
 $cache = Cache.new(dir: 'site-scan')
 $verbose = ARGV.delete '--verbose'
 $skipresourcecheck = ARGV.delete '--noresource'
-$skipresourcecheck = true # TEMP until can get it working
 
 puts "Started: #{Time.now}"  # must agree with site-scan monitor