Uncaught ReferenceError: process is not defined about supabase-js HOT 21 CLOSED

🎉 This issue has been resolved in version 1.3.2 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

from supabase-js.

Updating to 1.3.2 did fix the problem for me.

from supabase-js.

🎉 This issue has been resolved in version 1.1.4 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

from supabase-js.

For another data point, I was running into a similar issue, and applied the fix you put up in supabase/auth-js#42 locally, and it resolved the issue.

from supabase-js.

thanks for the patience everyone! We're super busy, so a bit slow getting to some of these items. I'll close this, and feel free to reopen if you have any issues

And thanks for checking out Supabase ⚡

from supabase-js.

I think this may have been fixed by supabase/auth-js#37 What version of gotrue-js is it pulling in?

from supabase-js.

I updated supabase-js to 1.1.4 and checked that gotrue-js is 1.9.3 (wich is the case) and I'm still having this issue

from supabase-js.

OK thanks for the update @Penkie . Reopening to investigate ..

from supabase-js.

1. Create Angular project
2. Create angular service

Could you please set this up on codesandbox or give some instructions? I've never used angular, this would probably take me an hour to figure out. Even better if you can start ad branch with anangular-todo-list in our examples. That would be massively helpful for the community (but not necessary, I just need a way to debug this)

from supabase-js.

I tried with stackblitz, no problem I didn't have any errors. Tried also on codesandbox, loads of problems, maybe the dependencies not working. Anyway I created a repo with the Angular project inside : https://github.com/Penkie/supabase-js-debug-angular
You can check there
Note that I did not create any service (like I mentionned up there that you have to, but actually no you don't), so everything is on app.component.ts

from supabase-js.

For what it's worth, I'm having the same issue in a svelte/sapper app. For now, I'm bypassing the error with window.process = window.process || null in the page's script. It's not exactly ideal, but for now I can move on :)

I've setup a basic repro repo at https://github.com/laurentS/supabase-bug-repro-113
to reproduce:

• clone the repo
• yarn install
• yarn run dev
• open the landing page (should be http://localhost:3000) and watch the error message appear after a few milliseconds

All it takes is importing the supabase client (see the last commit in the main branch).
Hope this helps!

from supabase-js.

Looks like gotrue-js is using process.env, which makes it non-isomorphic. supabase/auth-js#37 moves the problem to the bundler (Rollup) instead, which wasn't able to match-replace process.env. So if you add "process": undefined in the Rollup replace plugin in laurentS's example above, the app works.

Can we maybe refactor gotrue-js to use e.g. default parameters when initializing the client instead?

from supabase-js.

Error report from @thorwebdev
our unpkg build (https://unpkg.com/@supabase/[email protected]/dist/umd/supabase.js) is erroring

from supabase-js.

Hey @soedirgo - just checking on this commit

particularly these lines:

    new webpack.DefinePlugin({
      'process.env.GOTRUE_URL': undefined,
      'process.env.AUDIENCE': undefined,
      'process.env.EXPIRY_MARGIN': undefined,
      'process.env.STORAGE_KEY': undefined,
    }),

Is there a webpack plugin that can fix this issue for browser builds? Maybe this?

from supabase-js.

Huh, I don't remember adding those... let me try that.

from supabase-js.

Hey @Penkie, can you update the supabase-js client and see if it works now? I tried @laurentS's Sapper app with the update and saw no issue.

Edit: my bad, it's still not working. Give me a moment.

from supabase-js.

So using the Webpack plugin above wouldn't fix it, because it only affects the browser build that's in dist/umd. So the change needs to be in the user's bundler config.

@kiwicopple is there a specific reason to use process.env in gotrue-js? Looks like it's only used as default options for the client. Otherwise I can make a quick PR to fix this.

from supabase-js.

is there a specific reason to use process.env in gotrue-js

That was just as a convenience, but looks like it's problematic. How are these usually handled in isomorphic libs? Using dotenv?

from supabase-js.

I don't think libraries in general use process.env, only apps. Maybe it's possible to use Webpack for all 3 builds, but that's an antipattern (bundling should be done by end-user apps, not libraries), and more trouble than it's worth, IMO.

from supabase-js.

OK , just pushing the new version - I'll keep it open this time until we've confirmed that it's resolved

from supabase-js.

Should be working now, when trying laurentS's example above make sure to yarn upgrade first. Let us know if it works for y'all!

from supabase-js.