payloadcms / payload

Payload is the open-source, fullstack Next.js framework, giving you instant backend superpowers. Get a full TypeScript backend and admin panel instantly. Use Payload as a headless CMS or for building powerful applications.
https://payloadcms.com
MIT License
23.11k stars 1.44k forks source link

Nodemailer Email Adapter: skipVerify=true does not skip verification #6789

Closed sroaam closed 1 month ago

sroaam commented 2 months ago

Link to reproduction

No response

Payload Version

3.0.0-beta.47

Node Version

20.10.0

Next.js Version

15.0.0-canary.29

Describe the Bug

The skipVerify option of the NodemailerAdapterArgs has exactly the opposite effect.

skipVerify: true > Verifies the nodemailer transport skipVerify: false > Does not verify the nodemailer transport

import { nodemailerAdapter } from "@payloadcms/email-nodemailer";
...
export default buildConfig({
...
  email: nodemailerAdapter({
      ...
      skipVerify: true,
      ...
  }),
...
}

Reproduction Steps

  1. Add the @payloadcms/email-nodemailerpackage to your Next.js/Payload project
  2. Configure the adapter according to the docs Nodemailer Email Adapter Docs
  3. Set the skipVerify option to true
  4. Run/Build your Next.js/Payload project

Result: Nodemailer transporter.verify will be executed

Adapters and Plugins

@payloadcms/email-nodemailer, nodemailer

denolfe commented 1 month ago

Closed by https://github.com/payloadcms/payload/pull/6790

github-actions[bot] commented 16 hours ago

This issue has been automatically locked. Please open a new issue if this issue persists with any additional detail.