Something I noticed when duplicating a logged-in session on 2 browser tabs. If I log out on the second tab and click around on the first tab, the behavior is 'interesting'.
I observed the cookie is cleared so the user session is gone as expected in both tabs
On the first tab: If I refresh the page from the profile page or one of its sub-pages, it redirects back to the home page and updates the nav bar properly as expected.
On the first tab: if I am on the profile page and click a sub-page (eg: change password) the body of the page is updated (redirected to home) but the nav bar is not updated and still shows the 'Logout' button. Any idea what's causing this difference in behavior?
I am still seeing the same behavior on the browser (logout button is still visible). I added a console.log on onMount and noticed it doesn't get printed when following this sequence of actions.
Not sure if related, I am using TS, and the IDE warns me on the line onMount(async () => {
that: Svelte: Argument of type () => Promise<() => void> is not assignable to parameter of type () => (() => any) | Promise<never>
Let me first say, thank you so much for this awesome article!
I am however wondering about the warning I keep getting:
Using the user object as returned from supabase.auth.getSession() or from some supabase.auth.onAuthStateChange() events could be insecure! This value comes directly from the storage medium (usually cookies on the server) and many not be authentic. Use supabase.auth.getUser() instead which authenticates the data by contacting the Supabase Auth server.
Any ideas how to remove this? All of my attempts have fallen flat.
Concerning the warning I recommend to follow Supabase recently updated official documentation. I did not have time to update this tutorial. But I guess you should be good as long as you call supabase.auth methods with supabase.auth.getUser() to check JWT validation.
Try this event.locals.safeGetSession, see the new documentation here
import{PUBLIC_SUPABASE_URL,PUBLIC_SUPABASE_ANON_KEY}from'$env/static/public'import{createServerClient}from'@supabase/ssr'importtype{Handle}from'@sveltejs/kit'exportconsthandle:Handle=async ({event,resolve})=>{event.locals.supabase=createServerClient(PUBLIC_SUPABASE_URL,PUBLIC_SUPABASE_ANON_KEY,{cookies:{get:(key)=>event.cookies.get(key),/**
* Note: You have to add the `path` variable to the
* set and remove method due to sveltekit's cookie API
* requiring this to be set, setting the path to an empty string
* will replicate previous/standard behaviour (https://kit.svelte.dev/docs/types#public-types-cookies)
*/set:(key,value,options)=>{event.cookies.set(key,value,{...options,path:'/'})},remove:(key,options)=>{event.cookies.delete(key,{...options,path:'/'})},},})/**
* Unlike `supabase.auth.getSession()`, which returns the session _without_
* validating the JWT, this function also calls `getUser()` to validate the
* JWT before returning the session.
*/event.locals.safeGetSession=async ()=>{const{data:{session},}=awaitevent.locals.supabase.auth.getSession()if (!session){return{session:null,user:null}}const{data:{user},error,}=awaitevent.locals.supabase.auth.getUser()if (error){// JWT validation has failedreturn{session:null,user:null}}return{session,user}}returnresolve(event,{filterSerializedResponseHeaders(name){returnname==='content-range'||name==='x-supabase-api-version'},})}
Great article, you are much more talented and focused than I am. I love the way you avoid the "infamous warnings" , very elegant but that's me. My only question is can getUser return null but also no error ? One q. Should I check for !user also? Wonder if you know anything about this?
Thank you very much. I still remember to be a total noob to programming. In my case it is like other skills, it takes time, focus and practice. But till today I do not consider myself to be talented.
Hi mate.
Do you have a public GitHub repo?
Want to start a SaaS project from it.
This will support multi users having their own separate sessions once deployed right?
Tried to do the session validation in the middleware on dotnet with Supabase but all my users ended up sharing the same session.
So I don't want to have to rethink all that and just get started with protected routes from your Svelte+supabase starter
Thanks again for the article. I am still not all the way there though. Everything works as expected in development but when I try to deploy on vercel, I get the error:
RollupError: "PUBLIC_SUPABASE_URL" is not exported by "virtual:$env/static/public", imported by "src/routes/+layout.ts".
exportasyncfunctionload({locals:{getSession}}){constsession=awaitgetSession();// if the user is already logged in return him to the home pageif (session){redirect(303,'/');}}
Each +page.server.js is checking if user is already looged in. Is there a way to check it in the +layout.server.js?
I am just discussing the update of the official documentaton with Supanase. This documentaton uses TS and can be found here: supabase.com/docs/guides/auth/serv...
Awesome article, thank you!
Have you tried doing OAuth? Any chance you could add this to the article too?
Thank you for your interest.
I definitely plan to look into OAuth as well.
Something I noticed when duplicating a logged-in session on 2 browser tabs. If I log out on the second tab and click around on the first tab, the behavior is 'interesting'.
I am still seeing the same behavior on the browser (logout button is still visible). I added a console.log on onMount and noticed it doesn't get printed when following this sequence of actions.
Not sure if related, I am using TS, and the IDE warns me on the line
onMount(async () => {that:
Svelte: Argument of type () => Promise<() => void> is not assignable to parameter of type () => (() => any) | Promise<never>Thank you for your efforts. I'll take a look!
Let me first say, thank you so much for this awesome article!
I am however wondering about the warning I keep getting:
Using the user object as returned from supabase.auth.getSession() or from some supabase.auth.onAuthStateChange() events could be insecure! This value comes directly from the storage medium (usually cookies on the server) and many not be authentic. Use supabase.auth.getUser() instead which authenticates the data by contacting the Supabase Auth server.Any ideas how to remove this? All of my attempts have fallen flat.
Also, an OAuth article would be awesome!!!
Thank you!!!
Hi, thank you for your thank you :)
Concerning the warning I recommend to follow Supabase recently updated official documentation. I did not have time to update this tutorial. But I guess you should be good as long as you call
supabase.authmethods withsupabase.auth.getUser()to check JWT validation.Try this
event.locals.safeGetSession, see the new documentation hereI Understand now and based on your example I was able to solve the issue. Again, THANK YOU!
I have just updated the tuutorial to refelect new ssr package changes. My code should also solve the warning issues.
Great article, you are much more talented and focused than I am. I love the way you avoid the "infamous warnings" , very elegant but that's me. My only question is can getUser return null but also no error ? One q. Should I check for !user also? Wonder if you know anything about this?
Thank you very much. I still remember to be a total noob to programming. In my case it is like other skills, it takes time, focus and practice. But till today I do not consider myself to be talented.
Awesome article, you have helped me so much as I am new to the JS world. Please do an OAuth one with Supabase+SK as well! Thanks!!
Glad to hear you did like the tutorial.
Hi mate.
Do you have a public GitHub repo?
Want to start a SaaS project from it.
This will support multi users having their own separate sessions once deployed right?
Tried to do the session validation in the middleware on dotnet with Supabase but all my users ended up sharing the same session.
So I don't want to have to rethink all that and just get started with protected routes from your Svelte+supabase starter
Hi, if you need session validation I recommend you this github.com/j4w8n/sveltekit-supabas... repo. For validation look into hooks.server.ts github.com/j4w8n/sveltekit-supabas...
file.
Supabase guys are working on introducing asymmetric JWTs so expect changes anyway.
Thank you for this post kvetoslanovak!
Works perfectly! 😁😍
Thanks again for the article. I am still not all the way there though. Everything works as expected in development but when I try to deploy on vercel, I get the error:
RollupError: "PUBLIC_SUPABASE_URL" is not exported by "virtual:$env/static/public", imported by "src/routes/+layout.ts".
Hi, thank you for yout thank you.
Concernig Vercel did you set up all your environment varaibles in Vercel? vercel.com/docs/projects/environme...
Great article! I'm new with sveltekit, Is there a form to avoid duplicate load function for session validation?
Hi, thank you.
I am not sure which duplicate load function do you mean? Could you be more specific, please?
Each +page.server.js is checking if user is already looged in. Is there a way to check it in the +layout.server.js?
I would not recommend using +layout.server.js for auth checking. See more details in this discussion
github.com/sveltejs/kit/issues/6315 or this video https://www.youtube.com/watch?v=UbhhJWV3bmI&ab_channel=Huntabyte
Conrening protected routes and redirects in one place you may follow this advice https://www.youtube.com/watch?v=K1Tya6ovVOI&ab_channel=Huntabyte and move the logic to src/hooks.server.js
just started a new project and decide to use these two; great article!
but I was wondering what the addUserprofileToUser function did?
Thank you, I have added the explanaton as a second edit in the bottom of the article.
Thanks for the tutorial. What would the app.d.ts file look like for those of us in love with typescript?
I am just discussing the update of the official documentaton with Supanase. This documentaton uses TS and can be found here: supabase.com/docs/guides/auth/serv...
In previous documentation for Supabase Auth Helpers there is src/app.d.ts supabase.com/docs/guides/auth/auth...
Thanks for the great article!
Pls share the ./utils/addUserprofileToUser.js file
Thank you, I am sharing this function in a second edit in the bottom of the article.