Merge branch 'fix-express-error-handle' of github.com:ut-code/CourseMate into make-it-ssg

Author: nakaterm

server/src/index.ts

@@ -20,37 +20,6 @@ const app = express();
 // https://expressjs.com/ja/api.html#app.settings.table  の query parser を参照。
 app.set("query parser", "simple");
 
-// I don't understand any of those
-// https://expressjs.com/en/guide/error-handling.html
-// https://qiita.com/nyandora/items/cd4f12eb62295c10269c
-// https://note.shiftinc.jp/n/n42b96d36f0cf
-// エラーハンドラを Express に管理させる。
-// TEMPORARY: will be replaced by hono's `onError`.
-app.use(async (err: unknown, _: unknown, res: unknown, next: unknown) => {
-  try {
-    if (typeof (err as Error)?.cause === "number") {
-      (res as Response)
-        .status((err as Error).cause as number)
-        .send((err as Error).message);
-    } else {
-      console.error(err);
-      (res as Response).status(500).send("Internal Error");
-    }
-    await (next as () => Promise<unknown>)();
-  } catch (err) {
-    console.log("[ERR] failed to handle error:", err);
-    try {
-      (res as Response).status(500).send("Internal error");
-    } catch (err) {
-      console.log("[ERR] failed to handle error twice:", err);
-    }
-  } finally {
-    try {
-      (res as Response).end("\n");
-    } catch {}
-  }
-});
-
 const port = process.env.PORT || 3000;
 const allowedOrigins = (
   process.env.CORS_ALLOW_ORIGINS || panic("env CORS_ALLOW_ORIGINS is missing")
@@ -91,6 +60,37 @@ app.use("/requests", requestsRoutes);
 app.use("/matches", matchesRoutes);
 app.use("/chat", chatRoutes);
 
+// I don't understand any of those
+// https://expressjs.com/en/guide/error-handling.html
+// https://qiita.com/nyandora/items/cd4f12eb62295c10269c
+// https://note.shiftinc.jp/n/n42b96d36f0cf
+// エラーハンドラを Express に管理させる。
+// TEMPORARY: will be replaced by hono's `onError`.
+app.use(async (err: unknown, _: unknown, res: unknown, next: unknown) => {
+  try {
+    if (typeof (err as Error)?.cause === "number") {
+      (res as Response)
+        .status((err as Error).cause as number)
+        .send((err as Error).message);
+    } else {
+      console.error(err);
+      (res as Response).status(500).send("Internal Error");
+    }
+    await (next as () => Promise<unknown>)();
+  } catch (err) {
+    console.log("[ERR] failed to handle error:", err);
+    try {
+      (res as Response).status(500).send("Internal error");
+    } catch (err) {
+      console.log("[ERR] failed to handle error twice:", err);
+    }
+  } finally {
+    try {
+      (res as Response).end("\n");
+    } catch {}
+  }
+});
+
 export function main() {
   // サーバーの起動
   const server = app.listen(port, () => {