fix: Access localStorage lazily to avoid DOMExceptions in constrained envs when localStorage is provided using options (#252)

Author: Andarist

package-lock.json

@@ -36,7 +36,7 @@
     "docs:json": "typedoc --json docs/spec.json --mode modules --includeDeclarations --excludeExternals"
   },
   "dependencies": {
-    "@supabase/gotrue-js": "^1.17.0",
+    "@supabase/gotrue-js": "^1.17.1",
     "@supabase/postgrest-js": "^0.33.3",
     "@supabase/realtime-js": "^1.1.3",
     "@supabase/storage-js": "^1.4.0"

package.json

@@ -11,7 +11,6 @@ const DEFAULT_OPTIONS = {
   autoRefreshToken: true,
   persistSession: true,
   detectSessionInUrl: true,
-  localStorage: globalThis.localStorage,
   headers: DEFAULT_HEADERS,
 }
 

src/SupabaseClient.ts

@@ -1,14 +1,8 @@
 import { GoTrueClient } from '@supabase/gotrue-js'
+import { SupabaseAuthClientOptions } from './types'
 
 export class SupabaseAuthClient extends GoTrueClient {
-  constructor(options: {
-    url?: string
-    headers?: { [key: string]: string }
-    detectSessionInUrl?: boolean
-    autoRefreshToken?: boolean
-    persistSession?: boolean
-    localStorage?: Storage
-  }) {
+  constructor(options: SupabaseAuthClientOptions) {
     super(options)
   }
 }

src/lib/SupabaseAuthClient.ts

@@ -1,5 +1,10 @@
+import { GoTrueClient } from '@supabase/gotrue-js'
 import { RealtimeClientOptions } from '@supabase/realtime-js'
 
+type GoTrueClientOptions = ConstructorParameters<typeof GoTrueClient>[0]
+
+export interface SupabaseAuthClientOptions extends GoTrueClientOptions {}
+
 export type SupabaseClientOptions = {
   /**
    * The Postgres schema which your tables belong to. Must be on the list of exposed schemas in Supabase. Defaults to 'public'.
@@ -24,7 +29,7 @@ export type SupabaseClientOptions = {
   /**
    * A storage provider. Used to store the logged in session.
    */
-  localStorage?: Storage
+  localStorage?: SupabaseAuthClientOptions['localStorage']
 
   /**
    * Options passed to the realtime-js instance