Improved it a tad bit more

src/hooks/useUser.tsx

@@ -10,6 +10,7 @@ export default function useUser({redirectTo = "", redirectIfFound = false} = {})
 	const {data: user, mutate: mutateUser, isLoading} = useSWR<User>("/api/user", fetcher);
 
 	useEffect(() => {
+		console.log("HERE!!!");
 		// if no redirect needed, just return (example: already on /dashboard)
 		// if user data not yet there (fetch in progress, logged in or not) then don't do anything yet
 		if (!redirectTo || !user) return;

src/pages/api/logout.ts

@@ -9,13 +9,7 @@ const auth = getAuth(app);
 export default withIronSessionApiRoute(logout, sessionOptions);
 
 async function logout(req: NextApiRequest, res: NextApiResponse) {
-	signOut(auth)
+	await auth.signOut();
-		.then(() => {
 	req.session.destroy();
-			res.status(200).json({ok: true});
+	res.status(200).json(undefined);
-		})
-		.catch(() => {
-			req.session.destroy();
-			res.status(500).json({ok: false});
-		});
 }

src/pages/api/user.ts

@@ -10,17 +10,17 @@ export default withIronSessionApiRoute(user, sessionOptions);
 async function user(req: NextApiRequest, res: NextApiResponse) {
 	if (req.session.user) {
 		if (!auth.currentUser) {
-			res.status(401).json({ok: false});
+			res.status(401).json(undefined);
 			return;
 		}
 
 		if (req.session.user.id === auth.currentUser.uid) {
-			res.status(401).json({ok: false});
+			res.status(401).json(undefined);
 			return;
 		}
 
 		res.json(req.session.user);
 	} else {
-		res.status(401).json({ok: false});
+		res.status(401).json(undefined);
 	}
 }