add error checking

apps/node/src/routes/proxy.ts

@@ -27,6 +27,7 @@ export default class ProxyRoute extends Route {
       return;
     }
     // TODO: handle rate limiting? and/or caching?
+    try {
       const response = await axios.get(url, {
         headers: {
           "Content-Type": "application/json",
@@ -59,5 +60,10 @@ export default class ProxyRoute extends Route {
 
       // Return the JSON response
       res.status(response.status).set(headers).json(data);
+    } catch (ex) {
+      res
+        .status(500)
+        .json(RouteMessages.internalServerError("Error fetching URL"));
+    }
   }
 }

apps/proxy/src/routes/proxy.ts

@@ -2,6 +2,8 @@ import { Request, Response } from "express";
 import { Route, RouteMessages } from "server";
 import { nodeManager } from "..";
 
+const IGNORED_PATHS = ["/favicon.ico"];
+
 export default class ProxyRoute extends Route {
   constructor() {
     super({ path: "/" });
@@ -13,6 +15,10 @@ export default class ProxyRoute extends Route {
       res.status(400).json(RouteMessages.badRequest("No URL provided"));
       return;
     }
+    if (IGNORED_PATHS.includes(url)) {
+      res.status(404).send("Not found");
+      return;
+    }
 
     const node = nodeManager.getRandomNode();
     if (!node) {
@@ -22,10 +28,19 @@ export default class ProxyRoute extends Route {
       return;
     }
 
+    try {
       const response = await node.fetch(url);
       const data = response.data;
 
+      if (response.status === 500) {
+        res.status(500).json(RouteMessages.internalServerError(data));
+        return;
+      }
+
       // Return the JSON response
       res.status(response.status).set(response.headers).json(data);
+    } catch (ex: any) {
+      res.status(500).json(RouteMessages.internalServerError(ex.message || ex));
+    }
   }
 }