refactor: logout triggers revalidation of / path, lets requirePageAut…

…h handle redirect

removes need for props on logout or redirect happening in the logout action.

actions/auth.ts

@@ -8,6 +8,7 @@ import { getServerSession, lucia } from '~/utils/auth';
 import { redirect } from 'next/navigation';
 import { generateIdFromEntropySize } from 'lucia';
 import { loginSchema } from '~/schemas/auth';
+import { revalidatePath } from 'next/cache';
 
 export async function signup(formData: FormData) {
   const username = formData.get('username') as string;
@@ -96,11 +97,8 @@ export async function login(formData: FormData) {
 
   return redirect('/dashboard');
 }
-/**
- * @param disableRedirect If true, does not redirect to the homepage after logging out
- */
 
-export async function logout(disableRedirect?: boolean) {
+export async function logout() {
   const { session } = await getServerSession();
   if (!session) {
     return {
@@ -117,9 +115,5 @@ export async function logout(disableRedirect?: boolean) {
     sessionCookie.attributes,
   );
 
-  if (disableRedirect) {
-    return;
-  }
-
-  return redirect('/');
+  revalidatePath('/');
 }

components/Feedback/SignOutModal.tsx

@@ -39,7 +39,7 @@ const SignOutModal = ({
           <AlertDialogCancel onClick={() => setOpenSignOutModal(false)}>
             Cancel
           </AlertDialogCancel>
-          <AlertDialogAction onClick={() => void logout(true)}>
+          <AlertDialogAction onClick={() => void logout()}>
             Sign Out and Hide Banner
           </AlertDialogAction>
         </AlertDialogFooter>