Remove extra console.log statements (#811)

- remove one added in screencaster
- also remove others that are outside logging system
- bump to 1.5.10

package.json

@@ -1,6 +1,6 @@
 {
   "name": "browsertrix-crawler",
-  "version": "1.5.9",
+  "version": "1.5.10",
   "main": "browsertrix-crawler",
   "type": "module",
   "repository": "https://github.com/webrecorder/browsertrix-crawler",

src/crawler.ts

@@ -2871,7 +2871,6 @@ function getDownloadResponse(req: HTTPRequest) {
       return resp;
     }
   } catch (e) {
-    console.log(e);
     // ignore
   }
 

src/replaycrawler.ts

@@ -285,7 +285,11 @@ export class ReplayCrawler extends Crawler {
     try {
       pageData = JSON.parse(page);
     } catch (e) {
-      console.log(page, e);
+      logger.error(
+        "Error parsing page data",
+        { data: page, ...formatErr(e) },
+        "replay",
+      );
       return;
     }
 

src/util/screencaster.ts

@@ -195,8 +195,6 @@ class ScreenCaster {
     this.transport = transport;
     this.transport.caster = this;
 
-    console.log("RATIO", ratio);
-
     if (ratio) {
       this.maxHeight = this.maxWidth / ratio;
     }

src/util/sitemapper.ts

@@ -304,13 +304,13 @@ export class SitemapReader extends EventEmitter {
         }
       } else if (!otherTags) {
         if (parsingUrl) {
-          console.warn("text in url, ignoring");
+          logger.debug("text in url, ignoring", {}, "sitemap");
         } else if (parsingUrlset) {
-          console.warn("text in urlset, ignoring");
+          logger.debug("text in urlset, ignoring", {}, "sitemap");
         } else if (parsingSitemap) {
-          console.warn("text in sitemap, ignoring");
+          logger.debug("text in sitemap, ignoring", {}, "sitemap");
         } else if (parsingSitemapIndex) {
-          console.warn("text in sitemapindex, ignoring");
+          logger.debug("text in sitemapindex, ignoring", {}, "sitemap");
         }
       }
     };