From 0b0075e819ea9ace44e8c9cd2c2f2c975cda5477 Mon Sep 17 00:00:00 2001
From: Brent Vollebregt <brent.vollebregt@gmail.com>
Date: Sat, 18 May 2024 12:08:04 +1200
Subject: [PATCH] Stop logging Genius 404s

---
 src/api/genius.ts | 29 ++++++++++++++++++++++++++---
 src/api/lrclib.ts |  4 +++-
 2 files changed, 29 insertions(+), 4 deletions(-)

diff --git a/src/api/genius.ts b/src/api/genius.ts
index 718511a..5277d6c 100644
--- a/src/api/genius.ts
+++ b/src/api/genius.ts
@@ -151,7 +151,13 @@ export const getLyricsForPath = async (geniusPath: string): Promise<IFoundLyrics
   const requestUrl = `https://genius.com${geniusPath}`;
 
   try {
-    const response = await axios.get<string>(requestUrl);
+    const response = await axios.get<string>(requestUrl, {
+      validateStatus: status => status === 200 || status === 404
+    });
+
+    if (response.status === 404) {
+      return null;
+    }
 
     const html = response.data;
     const $ = cheerio.load(html); // Load in the page
@@ -171,8 +177,25 @@ export const getLyricsForPath = async (geniusPath: string): Promise<IFoundLyrics
       syncedLyrics: null
     } as IFoundLyrics;
   } catch (e) {
-    console.error(`Failed to call '${requestUrl}'`);
-    console.error(e);
+    // Anything non-200 or 404 is considered an error
+    console.warn(`Failed to call '${requestUrl}'`);
+
+    if (e instanceof Error && e.stack !== undefined) {
+      console.warn(e.stack);
+    }
+
+    if (axios.isAxiosError(e)) {
+      if (e.response !== undefined) {
+        console.log(`Response: HTTP${e.response.status} ${e.response.statusText}`);
+        console.log(`Response headers: ${JSON.stringify(e.response.headers)}`);
+        console.log(`Response data: ${JSON.stringify(e.response.data)}`);
+      } else {
+        console.log("No response");
+      }
+    } else {
+      console.warn(e);
+    }
+
     return null;
   }
 };
diff --git a/src/api/lrclib.ts b/src/api/lrclib.ts
index 3ad6ff4..d86d0c9 100644
--- a/src/api/lrclib.ts
+++ b/src/api/lrclib.ts
@@ -69,7 +69,7 @@ export async function getLyrics(
       attribution: LRCLIB_BASE_URL
     } as IFoundLyrics;
   } catch (e) {
-    // Can include 404 and anything else non-2xx
+    // Anything non-200 or 404 is considered an error
     console.warn(`Failed to call '${requestUrl}'`);
 
     if (e instanceof Error && e.stack !== undefined) {
@@ -81,6 +81,8 @@ export async function getLyrics(
         console.log(`Response: HTTP${e.response.status} ${e.response.statusText}`);
         console.log(`Response headers: ${JSON.stringify(e.response.headers)}`);
         console.log(`Response data: ${JSON.stringify(e.response.data)}`);
+      } else {
+        console.log("No response");
       }
     } else {
       console.warn(e);