-
-
Notifications
You must be signed in to change notification settings - Fork 172
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
NextAuth >= v4.23.0 Breaking change: - Package subpath './core' is not defined by "exports" #514
Comments
Exact same problem here. My workaround also was to pin So |
same problem here... |
faced the same problem, reverting to |
- downgraded next-auth (sidebase/nuxt-auth#514)
same problem here... |
@SherifAugustus where exactly should I revert the next-auth version? |
@kennedyngigi4 just install the specific |
I had same issue and |
same issue exists in [email protected], but it works with [email protected]. |
Same problem |
4.23.2 same issue. |
Hello,I'm also having this problem. What does pinning mean and how to do that? Thanks |
@dingkai031 in the After removing it, make sure to run an install command again so that the pinned version is the one installed |
Hi @sduduzog and everyone else! I just tested the newest version of NextAuth (4.23.2) and it seems to have resolved the issue! You should be able to use the newest version again, as they undid their change with the exports! Therefore I will be closing this issue! |
#Hi @sduduzog and everyone else! I just tested the newest version of NextAuth (4.23.2) and it seems to have resolved the issue! You should be able to use the newest version again, as they undid their change with the exports! Therefore I will be closing this issue! Hey just tested this with 4.23.2 and the bug is still there. This shouldn't be closed yet. |
Yes. The error remains |
Too bad, Ill keep this open. Maybe someone wants to open a PR on the offical NextAuth repo or pressure them more into reverting this change! I had mentioned it before in the commit comments, but received no answer... |
NextAuth
>= v4.23.0 Breaking change: - Package subpath './core' is not defined by "exports"
NextAuth
>= v4.23.0 Breaking change: - Package subpath './core' is not defined by "exports"
Hi everyone. Thank you for regularly checking into this issue! However, this problem will sadly not be reverted or fixed by itself. After some investigation, @BracketJohn and I have identified how we need to rewrite NuxtAuth to be compatible with NextAuth's new system architecture. Once we have done this upgrade, we will not be compatible with NextAuth versions under 4.23. We are continuing to look into how we can best integrate the newer versions of NextAuth, without making any major code changes in our module, to ensure for a smooth transfer. |
@zoey-kaiser is there a timeline for the upgrade? Would the upgrade cause major backwards incompatible changes In the way Nuxt Auth is used? I.e. I want to implement Nuxt Auth in a new app, but if the fixed version is soon to come out and works mainly differently, I might focus my time elsewhere for the time being. Also, as a sidenote, when having
|
I think |
Sadly we are kind of stuck between two (non optional) solutions here. 1.) NextAuth was updated to partially use the new authjs infrastructure, which is why the breaking change happened. Rewriting our module to use their new architecture, would be a good amount of work and would need to be repeated once authjs is completely out (as we can then remove NextAuth completely)
The good news about this: It does not matter for NuxtAuth. The vulnerability has to do with the middleware provided by NextAuth. As we provide our own middleware and do not use theirs, this vulnerability does not apply to NuxtAuth! |
just to say other than this issue also when I used [email protected] and the baseURL set in auth object inside nuxt.config.ts I still get the default url which is http://127.0.0.1:3000/api/auth/signin/credentials for signinUrl and callbackURL rather then the baseURL set by myself. [email protected] works okay! |
@yuelongh Same config here. I resolved by setting |
Same issue |
same issue |
yep same, what a bummer! thanks nuxt-auth team for your work! |
this broke our site |
Hi there, I added these two lines in the "./core": {
"types": "./core/index.d.ts",
"default": "./core/index.js"
}, I know this won't work in production and even in development it's only a temporary fix. I'm on |
Same issue. |
Includes pinning 'next-auth' (with an _e_) at 4.21.1 as a workaround for "core" export issue documented here: sidebase/nuxt-auth#514.
It felt a bit weird to have to install next-auth when @sidebase/nuxt-auth includes it, adding the following to my package.json seems to work to pin the version (using npm):
|
Do we have any possible update on this matter? Locking version to |
Hi everyone! We are now beginning this migration, for more information please keep an eye on #673. As this issue will be fixed through this migration, I will now close this issue. Thank you for all the hard work investigating! |
❌ Tested today and
✅ Working with |
Thanks for checking this. We also use |
🚀 Here’s a compatible solution that works after extensive research:
|
Environment
Reproduction
n/a
Describe the bug
When doing a clean install with no package-lock.json, the next-auth package installed along with nuxt-auth module seem to have breaking changes in import namespaces used by the nuxtAuthHandler file.
Additional context
A current workaround is pinning next-auth to v4.22.5
Logs
The text was updated successfully, but these errors were encountered: