supabase / auth-helpers

A collection of framework specific Auth utilities for working with Supabase.

Home Page:https://supabase.github.io/auth-helpers/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[supabase/ssr] Calling `createBrowserClient` multiple times overriding cached client options

bombillazo opened this issue · comments

Bug report

  • I confirm this is a bug with Supabase, not with my own application.
  • I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug

We're using the createBrowserClient function to access the supabase client all across our frontend logic, where hooks are not available to get the client using the useSupabaseClient hook.

We noticed that if we do not pass the exact same options across the clients when we call this function, the existing client-cached options are overridden by other "instantiations." For example, we used the createBrowserClient function in our root app context, and in our auth helper function. In one, we defined common options to override and control, while in the other, we passed no options since we are leveraging the singleton pattern and expect the cached instance to keep the options.

The options we are changing specifically are:

{
  auth: {
    storage: {}
  },
  cookies: {}
}

This is causing issues where we are indeed using the same client but with reconfigured options since the function is not correctly merging the user-defined options with the default options. This is causing all sorts of havoc in the app due to inconsistent client behavior.

To Reproduce

Steps to reproduce the behavior, please provide code snippets or a repository:

  1. Create a browser client using the createBrowserClient and pass it specific client options (like custom get/set cookies)
  2. In another page or function, call the createBrowserClient with no options to get a second client
  3. Call the page/function where the second client is used; check the singleton client options, and they will not be the same.

Expected behavior

In theory, since we are using singleton = true by default and are not modifying it, we get the same instance. And we do, but the options have defaulted back since the second createBrowserClient call had no options, and it internally wrote over the original options.

Screenshots

If applicable, add screenshots to help explain your problem.

System information

  • OS: [e.g. macOS, Windows]
  • Browser (if applies) [e.g. chrome, safari]
  • Version of supabase-js: [e.g. 6.0.2]
  • Version of Node.js: [e.g. 10.10.0]

Additional context

Add any other context about the problem here.