Missing `await` in samples cause node app to crash? about supabase HOT 8 CLOSED

Hi @jonlivingstone - Thank you for opening this issue, please feel free to open a PR! 👍

from supabase.

check out the pull request add the await .

return await resolve(event, {
filterSerializedResponseHeaders(name) {
/**

• Supabase libraries use the content-range and x-supabase-api-version
• headers, so we need to tell SvelteKit to pass it through. */ return name === 'content-range' || name === 'x-supabase-api-version' }, })

from supabase.

There are a few more occurences I have changed.
I have submitted a PR as suggested.
#26848

Thank you!

from supabase.

@jonlivingstone can you assign this issue to me?

from supabase.

Hi @Anubhavbaranwal - we don't have an assignment process. Feel free to open a pr though, thanks!

from supabase.

Hi @Anubhavbaranwal, thanks for your interest in helping out! Just a warning that we don't believe this to be the cause of the issue (you can see the linked PR above for the discussion). If you want to try reproducing to discover the root cause, you're more than welcome! If you want a simpler first issue, feel free to submit a PR for another issue in the help wanted or good first issue sections!

from supabase.

@Anubhavbaranwal I would suggest to have a minimal repro ready so the owners can root cause and reproduce the problem. Otherwise it won't get merged (and for good reason). I was not successful in doing that.
I thought that was the issue and it is apparently something else. I believe there was something in my code that messed up the page lifecycle and the await was only accidentally helping, if anything.

from supabase.

Hi all, We’re going to close this one due to inactivity. Please feel free to reopen if you’re still having issues here or open a new issue.

from supabase.