Fix: wrap async method in try catch (#811)

isomerpages · Jun 22, 2023 · 32ef92c · 32ef92c
1 parent 60f3331
commit 32ef92c
Showing 1 changed file with 20 additions and 16 deletions.
diff --git a/src/middleware/authentication.ts b/src/middleware/authentication.ts
@@ -30,21 +30,25 @@ export class AuthenticationMiddleware {
     next: NextFunction
   ) {
     const { cookies, originalUrl: url, session } = req
-    const {
-      isomerUserId,
-      email,
-      ...rest
-    } = await this.authenticationMiddlewareService.verifyAccess({
-      cookies,
-      url,
-      userInfo: session.userInfo,
-    })
-    const userSessionData = new UserSessionData({
-      isomerUserId,
-      email,
-      ...rest,
-    })
-    res.locals.userSessionData = userSessionData
-    return next()
+    try {
+      const {
+        isomerUserId,
+        email,
+        ...rest
+      } = await this.authenticationMiddlewareService.verifyAccess({
+        cookies,
+        url,
+        userInfo: session.userInfo,
+      })
+      const userSessionData = new UserSessionData({
+        isomerUserId,
+        email,
+        ...rest,
+      })
+      res.locals.userSessionData = userSessionData
+      return next()
+    } catch (err) {
+      return next(err)
+    }
   }
 }