fix(auth): fix no session bug

profilecity · Sep 10, 2024 · 8dac2bb · 8dac2bb
1 parent 5839dfa
commit 8dac2bb
Show file tree

Hide file tree

Showing 6 changed files with 8 additions and 7 deletions.
diff --git a/app/components/admin/MemberCard.vue b/app/components/admin/MemberCard.vue
@@ -29,7 +29,7 @@ const onRemove = () => {
         title="Remove Member?"
         confirmLabel="Remove"
         content="Member's access will be immediatly disabled. No actions performed by member will be reverted."
-        v-if="profile.id != member.id"
+        v-if="profile?.id != member.id"
       >
         <template #input="{ open }">
           <InputButton variant="destructive" size="icon" @click="open">

diff --git a/app/composables/auth.ts b/app/composables/auth.ts
@@ -1,7 +1,7 @@
 export function useAuth() {
   const session = useSessionState();
-  const isSignedIn = computed(() => !!session.value.profile);
-  const profile = computed(() => session.value.profile);
+  const isSignedIn = computed(() => !!session.value?.profile);
+  const profile = computed(() => session.value?.profile);
 
   return {
     session,

diff --git a/app/composables/state.ts b/app/composables/state.ts
@@ -2,7 +2,7 @@ import { type CareerSiteConfig, type SEOConfig } from '~~/shared/schemas/setting
 import type { Session } from '~~/shared/types/profile-types';
 
 export function useSessionState() {
-  return useState<Session>('oauth_session');
+  return useState<Session | null>('oauth_session');
 }
 
 export function useRemoteAssetBaseState() {

diff --git a/app/middleware/admin-auth.ts b/app/middleware/admin-auth.ts
@@ -6,7 +6,7 @@ export default defineNuxtRouteMiddleware(async (to) => {
     return redirectToLogin(to.fullPath);
   }
 
-  if (!profile.value.isAdmin) {
+  if (!profile.value?.isAdmin) {
     return navigateTo('/');
   }
 });
diff --git a/app/pages/admin/dashboard.vue b/app/pages/admin/dashboard.vue
@@ -15,7 +15,7 @@ const { profile } = useAuth();
 <template>
   <div class="w-full max-w-9xl mx-auto">
     <section class="sticky top-0 bg-white p-4 border-b border-zinc-200">
-      <div class="text-xl font-bold text-zinc-900 font-noto">👋🏽 Hello, {{ profile.firstName }}</div>
+      <div class="text-xl font-bold text-zinc-900 font-noto">👋🏽 Hello, {{ profile?.firstName }}</div>
       <div class="text-sm text-zinc-500">Track your hiring activities. You are almost there!</div>
       <div class="flex items-center mt-4">
         <InputButton as="NuxtLink" variant="secondary" to="/admin/postings/new">

diff --git a/server/api/userinfo.ts b/server/api/userinfo.ts
@@ -14,7 +14,8 @@ export default defineEventHandler(async (event) => {
 
     return session;
   } catch (e) {
-    if (IS_DEV) {
+    // @ts-expect-error
+    if (e.statusCode !== 401) {
       console.error(e);
     }
     throw e;