fix: update cookie lifetime. (#34)

* fix: update cookie lifetime. * chore: update changelog.
supabase · Mar 9, 2022 · 80173be · 80173be
1 parent 2f800f6
commit 80173be
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # CHANGELOG
 
+## 1.2.2 - 2022-03-09
+
+- [#34](https://github.com/supabase-community/supabase-auth-helpers/pull/34): fix: update cookie lifetime default from 8h to 7 days.
+
 ## 1.2.1 - 2022-03-09
 
 - [#33](https://github.com/supabase-community/supabase-auth-helpers/pull/33): fix: merge cookieOptions correctly.

diff --git a/src/nextjs/utils/getUser.ts b/src/nextjs/utils/getUser.ts
@@ -4,7 +4,7 @@ import {
  NextApiResponse
 } from 'next';
 import { User, createClient } from '@supabase/supabase-js';
-import { CookieOptions } from '../types';
+import { CookieOptions, ApiError } from '../types';
 import { setCookies } from '../../shared/utils/cookies';
 import { COOKIE_OPTIONS } from '../../shared/utils/constants';
 import { jwtDecoder } from '../../shared/utils/jwt';
@@ -22,7 +22,7 @@ export default async function getUser(
  | GetServerSidePropsContext
  | { req: NextApiRequest; res: NextApiResponse },
  options: GetUserOptions = {}
-): Promise<{ user: User | null; accessToken: string | null }> {
+): Promise<{ user: User | null; accessToken: string | null; error?: string }> {
  try {
  if (
  !process.env.NEXT_PUBLIC_SUPABASE_URL ||
@@ -90,7 +90,8 @@ export default async function getUser(
  return { user: user!, accessToken: access_token };
  }
  } catch (e) {
- console.log('Error getting user:', e);
- return { user: null, accessToken: null };
+ const error = e as ApiError;
+ console.log('Error getting user:', error);
+ return { user: null, accessToken: null, error: error.message };
  }
 }
diff --git a/src/shared/utils/constants.ts b/src/shared/utils/constants.ts
@@ -1,6 +1,6 @@
 export const COOKIE_OPTIONS = {
  name: 'sb',
- lifetime: 60 * 60 * 8,
+ lifetime: 7 * 24 * 60 * 60, // 7 days
  domain: '',
  path: '/',
  sameSite: 'lax'