Revert "fix error with errors"

This reverts commit 7601a1d4bf.

src/cronjob/downvoteSegmentArchiveJob.ts

@@ -29,7 +29,7 @@ export const archiveDownvoteSegment = async (dayLimit: number, voteLimit: number
 
     } catch (err) {
         Logger.error("Execption when insert segment in archivedSponsorTimes");
-        Logger.error(err);
+        Logger.error(err as string);
         return 1;
     }
 
@@ -47,7 +47,7 @@ export const archiveDownvoteSegment = async (dayLimit: number, voteLimit: number
 
     } catch (err) {
         Logger.error("Execption when deleting segment in sponsorTimes");
-        Logger.error(err);
+        Logger.error(err as string);
         return 1;
     }
 

src/routes/addUnlistedVideo.ts

@@ -23,7 +23,7 @@ export function addUnlistedVideo(req: Request, res: Response): Response {
         const timeSubmitted = Date.now();
         db.prepare("run", `INSERT INTO "unlistedVideos" ("videoID", "year", "views", "channelID", "timeSubmitted") values (?, ?, ?, ?, ?)`, [videoID, year, views, channelID, timeSubmitted]);
     } catch (err) {
-        Logger.error(err);
+        Logger.error(err as string);
         return res.sendStatus(500);
     }
 

src/routes/dumpDatabase.ts

@@ -65,10 +65,8 @@ function removeOutdatedDumps(exportPath: string): Promise<void> {
 
         // read files in export directory
         fs.readdir(exportPath, async (err: any, files: string[]) => {
-            if (err) {
+            if (err) Logger.error(err);
-                Logger.error(err);
+            if (err) return resolve();
-                return resolve();
-            }
 
             files.forEach(file => {
                 // we only care about files that start with "<tablename>_" and ends with .csv

src/routes/getIsUserVIP.ts

@@ -22,7 +22,7 @@ export async function getIsUserVIP(req: Request, res: Response): Promise<Respons
             vip: vipState,
         });
     } catch (err) {
-        Logger.error(err);
+        Logger.error(err as string);
         return res.sendStatus(500);
     }
 }

src/routes/getLockCategories.ts

@@ -25,7 +25,7 @@ export async function getLockCategories(req: Request, res: Response): Promise<Re
             categories
         });
     } catch (err) {
-        Logger.error(err);
+        Logger.error(err as string);
         return res.sendStatus(500);
     }
 }

src/routes/getLockCategoriesByHash.ts

@@ -54,7 +54,7 @@ export async function getLockCategoriesByHash(req: Request, res: Response): Prom
         // merge all locks
         return res.send(mergeLocks(lockedRows));
     } catch (err) {
-        Logger.error(err);
+        Logger.error(err as string);
         return res.sendStatus(500);
     }
 }

src/routes/getSkipSegments.ts

@@ -80,7 +80,7 @@ async function getSegmentsByVideoID(req: Request, videoID: VideoID, categories:
         return segments;
     } catch (err) {
         if (err) {
-            Logger.error(err);
+            Logger.error(err as string);
             return null;
         }
     }
@@ -125,8 +125,10 @@ async function getSegmentsByHash(req: Request, hashedVideoIDPrefix: VideoIDHash,
 
         return segments;
     } catch (err) {
-        Logger.error(err);
+        if (err) {
-        return null;
+            Logger.error(err as string);
+            return null;
+        }
     }
 }
 

src/routes/getUsername.ts

@@ -28,7 +28,7 @@ export async function getUsername(req: Request, res: Response): Promise<Response
             });
         }
     } catch (err) {
-        Logger.error(err);
+        Logger.error(err as string);
         return res.sendStatus(500);
     }
 }

src/routes/getViewsForUser.ts

@@ -26,7 +26,7 @@ export async function getViewsForUser(req: Request, res: Response): Promise<Resp
             return res.sendStatus(404);
         }
     } catch (err) {
-        Logger.error(err);
+        Logger.error(err as string);
         return res.sendStatus(500);
     }
 }

src/routes/postLockCategories.ts

@@ -68,7 +68,7 @@ export async function postLockCategories(req: Request, res: Response): Promise<s
             await db.prepare("run", `INSERT INTO "lockCategories" ("videoID", "userID", "category", "hashedVideoID", "reason") VALUES(?, ?, ?, ?, ?)`, [videoID, userID, category, hashedVideoID, reason]);
         } catch (err) {
             Logger.error(`Error submitting 'lockCategories' marker for category '${category}' for video '${videoID}'`);
-            Logger.error(err);
+            Logger.error(err as string);
             res.status(500).json({
                 message: "Internal Server Error: Could not write marker to the database.",
             });
@@ -89,7 +89,7 @@ export async function postLockCategories(req: Request, res: Response): Promise<s
                     [reason, userID, videoID, category]);
             } catch (err) {
                 Logger.error(`Error submitting 'lockCategories' marker for category '${category}' for video '${videoID}'`);
-                Logger.error(err);
+                Logger.error(err as string);
                 res.status(500).json({
                     message: "Internal Server Error: Could not write marker to the database.",
                 });

src/routes/postPurgeAllSegments.ts

@@ -29,7 +29,7 @@ export async function postPurgeAllSegments(req: Request, res: Response): Promise
         await db.prepare("run", `UPDATE "sponsorTimes" SET "hidden" = 1 WHERE "videoID" = ?`, [videoID]);
 
     } catch (err) {
-        Logger.error(err);
+        Logger.error(err as string);
         return res.sendStatus(500);
     }
     return res.sendStatus(200);

src/routes/postSegmentShift.ts

@@ -92,7 +92,7 @@ export async function postSegmentShift(req: Request, res: Response): Promise<Res
             }
         }
     } catch (err) {
-        Logger.error(err);
+        Logger.error(err as string);
         return res.sendStatus(500);
     }
 

src/routes/postSkipSegments.ts

@@ -679,7 +679,7 @@ export async function postSkipSegments(req: Request, res: Response): Promise<Res
             });
         }
     } catch (err) {
-        Logger.error(err);
+        Logger.error(err as string);
         return res.sendStatus(500);
     }
 

src/routes/setUsername.ts

@@ -80,7 +80,7 @@ export async function setUsername(req: Request, res: Response): Promise<Response
 
         return res.sendStatus(200);
     } catch (err) {
-        Logger.error(err);
+        Logger.error(err as string);
         return res.sendStatus(500);
     }
 }

src/routes/voteOnSponsorTime.ts

@@ -458,7 +458,7 @@ export async function voteOnSponsorTime(req: Request, res: Response): Promise<Re
         }
         return res.status(finalResponse.finalStatus).send(finalResponse.finalMessage ?? undefined);
     } catch (err) {
-        Logger.error(err);
+        Logger.error(err as string);
         return res.status(500).json({error: "Internal error creating segment vote"});
     }
 }

src/utils/logger.ts

@@ -53,20 +53,20 @@ class Logger {
         }
     }
 
-    error(str: string | Error) {
+    error(str: string) {
         this.log(LogLevel.ERROR, str);
     }
-    warn(str: string | Error) {
+    warn(str: string) {
         this.log(LogLevel.WARN, str);
     }
-    info(str: string | Error) {
+    info(str: string) {
         this.log(LogLevel.INFO, str);
     }
-    debug(str: string | Error) {
+    debug(str: string) {
         this.log(LogLevel.DEBUG, str);
     }
 
-    private log(level: LogLevel, str: string | Error) {
+    private log(level: LogLevel, str: string) {
         if (!this._settings[level]) {
             return;
         }

src/utils/youtubeApi.ts

@@ -20,7 +20,7 @@ export class YouTubeAPI {
                     return { err: null, data: JSON.parse(data) };
                 }
             } catch (err) {
-                return { err, data: null };
+                return { err: err as string | boolean, data: null };
             }
         }
 
@@ -45,7 +45,7 @@ export class YouTubeAPI {
                 return { err: result.statusText, data: null };
             }
         } catch (err) {
-            return {err, data: null};
+            return {err: err as string | boolean, data: null};
         }
     }
 }