Skip to content
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

Change from NextAuth v4 syntax to v5 syntaxes #598

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JulianJorgensen
Copy link

Here's initial draft of pull request. Apologies for format issues.

Copy link

vercel bot commented Nov 3, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
next-intl-example-next-13 ❌ Failed (Inspect) Nov 3, 2023 3:22pm

Copy link

vercel bot commented Nov 3, 2023

@JulianJorgensen is attempting to deploy a commit to the next-intl Team on Vercel.

A member of the Team first needs to authorize it.

@amarincolas
Copy link

amarincolas commented Nov 7, 2023

Apparently, this does not work anymore

const authMiddleware = auth(
  (req) => intlMiddleware(req),
  {
    callbacks: {
      authorized: ({token}) => token != null
    },
    pages: {
      signIn: '/login'
    }
  }
);

@DiegoGonzalezCruz
Copy link

I'm having issues with this... any ideas?

@alessandrojcm
Copy link

I got this working using this workaround. This is necessary due to a bug in next-auth itself. So this may need to be parked until that bug is resolved.

@amannn
Copy link
Owner

amannn commented Jul 8, 2024

@ScreamZ shared his implementation here: #1170. Might be worth checking that out when we continue with this PR.

Also #899 provides a potential implementation.

For context, I didn't look further into this yet as I'm waiting for Auth.js v5 stable to be released. I'd like to avoid keeping up to date with their beta versions, as long as the API might still change.

Many thanks to everyone who has already looked into this and provided help! 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants