Fix 500 if multi_auth cookie missing

stackernews · Nov 11, 2024 · 931cb64 · 931cb64
1 parent 5e08e03
commit 931cb64
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/pages/api/signout.js b/pages/api/signout.js
@@ -36,9 +36,9 @@ export default (req, res) => {
   cookies.push(cookie.serialize(`multi_auth.${userId}`, '', { ...cookieOptions, expires: 0, maxAge: 0 }))
 
   // update multi_auth cookie and check if there are more accounts available
-  const oldMultiAuth = b64Decode(req.cookies.multi_auth)
-  const newMultiAuth = oldMultiAuth.filter(({ id }) => id !== Number(userId))
-  if (newMultiAuth.length === 0) {
+  const oldMultiAuth = req.cookies.multi_auth ? b64Decode(req.cookies.multi_auth) : undefined
+  const newMultiAuth = oldMultiAuth?.filter(({ id }) => id !== Number(userId))
+  if (!oldMultiAuth || newMultiAuth?.length === 0) {
     // no next account available. cleanup: remove multi_auth + pointer cookie
     cookies.push(cookie.serialize('multi_auth', '', { ...cookieOptions, httpOnly: false, expires: 0, maxAge: 0 }))
     cookies.push(cookie.serialize('multi_auth.user-id', '', { ...cookieOptions, httpOnly: false, expires: 0, maxAge: 0 }))