Merge remote-tracking branch 'origin/master'

Author: ZhiXin18

backend/index.ts

@@ -93,8 +93,9 @@ app.get("/geocode/:postcode", async (req: Request, res: Response) => {
 
 // catchall error handler with json response
 // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unused-vars
-app.use((err: any, _req: Request, res: Response, _next: NextFunction) => {
-  console.error(err);
+app.use((err: any, req: Request, res: Response, _next: NextFunction) => {
+  // dump error and on
+  console.error("Server error: %j Caused by: %j", err, req);
   // format error
   res.status(err.status || 500).json({
     message: err.message,

backend/services/routing.ts

@@ -127,6 +127,10 @@ export class RoutingSvc {
 
     // Handle response failure
     if (!response.ok) {
+      if(r["code"] === "NoRoute") {
+        // no route found
+        throw new NotFoundError("Not route found between given locations.");
+      }
       throw new Error(
         `OSRM ${url.pathname} request failed with error: ${r["code"]}: ${r["message"]}`,
       );