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

Removing Email provider was a breaking change #12443

Closed
idapartners opened this issue Dec 30, 2024 · 1 comment
Closed

Removing Email provider was a breaking change #12443

idapartners opened this issue Dec 30, 2024 · 1 comment
Labels
triage Unseen or unconfirmed by a maintainer yet. Provide extra information in the meantime.

Comments

@idapartners
Copy link

What is the improvement or update you wish to see?

The v5 Migration guide does not document that the email provider has been replaced by the Nodemailer provider and a bunch of service-specific providers.

Is there any context that might help us understand?

Figuring this out was fairly straightforward, because the configuration threw an error with the legacy "Email" provider, and the switch to Nodemailer in the configuration was simple. However, I lost an hour or so figuring out that calls to next-auth/react signIn were silently failing, and that, e.g.,

const auth = await signIn('email', {
        email: user.email,
        redirect: false
})

needed a s/email/nodemailer (or whatever service-specific email provider). This should probably be documented in the Migration section.

(also, maybe there should be a separate PR that this shouldn't silently fail?)

Does the docs page already exist? Please link to it.

https://authjs.dev/getting-started/migrating-to-v5

@idapartners idapartners added the triage Unseen or unconfirmed by a maintainer yet. Provide extra information in the meantime. label Dec 30, 2024
@balazsorban44
Copy link
Member

Hi, this should work as before, as the Nodemailer provider was introduced in a backward-compatible way. (Nodemailer is the same as the Email provider, BTW, so it just became a re-export. See: https://github.com/nextauthjs/next-auth/blob/main/packages/core/src/providers/email.ts/#L18)

Please open a bug report where the Email provider stopped working for you. 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage Unseen or unconfirmed by a maintainer yet. Provide extra information in the meantime.
Projects
None yet
Development

No branches or pull requests

2 participants