-
-
Notifications
You must be signed in to change notification settings - Fork 232
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cookieOption field "name" is not correctly working #717
Comments
This is blocking a migration from @supabase/auth-helpers-sveltekit (where the name option works) to ssr (where the name option no longer works). I can confirm that the name option has no effect on the cookie used for auth data. |
I could be mistaken but #730 change applies to the serverClient but not browserClients which will still use Apparently this did exist at one point #677 but was reverted for unclear reason(s). |
Any reason it got reverted? I really need this |
as a workaround, you need to manually set the storageKey and the cookie.name to the same value. or, don't use the beta ssr auth-helper |
Thanks for the help. what do you mean beta? afaik is the recommended way now? https://supabase.com/docs/guides/auth/server-side/migrating-to-ssr-from-auth-helpers Also, about setting the storageKey, you mean patch the library myself? I do not see any |
I can confirm that the issue lies within Supabase and not within my own application. I have thoroughly searched through the documentation, GitHub Discussions, and Discord for any relevant information. Description of the Bug: In my application, I utilized createPagesBrowserClient and set the cookieOption as shown below. However, despite this configuration, the stored cookie did not change its name:
I suspect that the bug may be related to commit hash f7e5c2d. Prior to this commit, the base cookie name passed to createClient of @supabase/supabase-js was derived from option.auth.storageKey. However, after this commit, the storageKey field was removed and no other changes were made. I suggest considering a revert of this commit or exploring alternative solutions. |
Bug report
Describe the bug
I used
createPagesBrowserClient
for my application and set cookieOption to the below but stored cookie didn't change its name.I suspect that f7e5c2d makes this bug. Base cookie name passed to
createClient
of@supabase/supabase-js
comes fromoption.auth.storageKey
before this commit, butstorageKey
field is removed and nothing other is changed after it. Please consider reverting it or take another solution.The text was updated successfully, but these errors were encountered: