这是indexloc提供的服务,不要输入任何密码
Skip to content

feat(nodemailer): update for v6 & v7 #73297

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

hkleungai
Copy link
Contributor

@hkleungai hkleungai commented Jul 20, 2025

Changes in v6

  • Update sendMail() signature in lib/mailer/index.d.ts to typecheck optional extra arguments.
  • Introduce @aws-sdk/client-ses, update SESTransport.MailSesOptions & SESTransport.Options.SES typing accordingly.
  • Add overload for SESTransport.Options.SES, which exists since many versions ago.

Changes in v7

  • Update sendMail() signature in lib/mailer/index.d.ts to typecheck optional extra arguments.
  • Introduce @aws-sdk/client-sesv2, update SESTransport.MailSesOptions & SESTransport.Options.SES typing accordingly.
  • Make SESTransport.Options.SES.sesClient & SESTransport.Options.SES.SendEmailCommand required, as they are in the source code.
  • Remove a lot of no-longer-existed methods & attributes from SESTransport class.

Commits:

  • feat(nodemailer): split v6 & v7
  • feat(nodemailer): [v6] introduce aws-sdk for ses-transport
  • feat(nodemailer): [v7] introduce aws-sdk for ses-transport

Please fill in this template.

Select one of these and delete the others:

If changing an existing definition:

  • Provide a URL to documentation or source code which provides context for the suggested changes: <>
  • If this PR brings the type definitions up to date with a new version of the JS library, update the version number in the package.json.

@typescript-bot
Copy link
Contributor

typescript-bot commented Jul 20, 2025

@hkleungai Thank you for submitting this PR!

This is a live comment that I will keep updated.

1 package in this PR

Code Reviews

Because this is a widely-used package, a DT maintainer will need to review it before it can be merged.

You can test the changes of this PR in the Playground.

Status

  • ✅ No merge conflicts
  • ✅ Continuous integration tests have passed
  • 🕐 Most recent commit is approved by a DT maintainer

Once every item on this list is checked, I'll ask you for permission to merge and publish the changes.

Inactive

This PR has been inactive for 8 days.


Diagnostic Information: What the bot saw about this PR
{
  "type": "info",
  "now": "-",
  "pr_number": 73297,
  "author": "hkleungai",
  "headCommitOid": "fb7d804af8b4210b87da85c0c7bfa26d83c2d1a4",
  "mergeBaseOid": "1eeea8b520d1e037fd4183cc20c176c30af7b7b3",
  "lastPushDate": "2025-07-20T04:07:44.000Z",
  "lastActivityDate": "2025-07-20T04:07:44.000Z",
  "hasMergeConflict": false,
  "isFirstContribution": false,
  "tooManyFiles": false,
  "hugeChange": false,
  "popularityLevel": "Critical",
  "pkgInfo": [
    {
      "name": "nodemailer",
      "kind": "edit",
      "files": [
        {
          "path": "types/nodemailer/.npmignore",
          "kind": "package-meta-ok"
        },
        {
          "path": "types/nodemailer/lib/mailer/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/lib/ses-transport/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/nodemailer-tests.ts",
          "kind": "test"
        },
        {
          "path": "types/nodemailer/package.json",
          "kind": "package-meta-ok"
        },
        {
          "path": "types/nodemailer/v3/.eslintrc.json",
          "kind": "package-meta-ok"
        },
        {
          "path": "types/nodemailer/v3/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v3/nodemailer-tests.ts",
          "kind": "test"
        },
        {
          "path": "types/nodemailer/v3/package.json",
          "kind": "package-meta-ok"
        },
        {
          "path": "types/nodemailer/v6/.npmignore",
          "kind": "package-meta-ok"
        },
        {
          "path": "types/nodemailer/v6/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/addressparser/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/base64/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/dkim/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/dkim/message-parser.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/dkim/relaxed-body.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/dkim/sign.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/fetch/cookies.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/fetch/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/json-transport/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/mail-composer/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/mailer/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/mailer/mail-message.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/mime-funcs/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/mime-funcs/mime-types.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/mime-node/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/mime-node/last-newline.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/qp/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/sendmail-transport/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/sendmail-transport/le-unix.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/sendmail-transport/le-windows.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/ses-transport/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/shared/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/smtp-connection/data-stream.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/smtp-connection/http-proxy-client.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/smtp-connection/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/smtp-pool/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/smtp-pool/pool-resource.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/smtp-transport/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/stream-transport/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/well-known/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/lib/xoauth2/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/nodemailer/v6/nodemailer-tests.ts",
          "kind": "test"
        },
        {
          "path": "types/nodemailer/v6/package.json",
          "kind": "package-meta-ok"
        },
        {
          "path": "types/nodemailer/v6/tsconfig.json",
          "kind": "package-meta-ok"
        }
      ],
      "owners": [
        "rogierschouten",
        "dex4er",
        "bioball"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Critical"
    }
  ],
  "reviews": [],
  "mainBotCommentID": 3093184738,
  "ciResult": "pass"
}

@typescript-bot
Copy link
Contributor

🔔 @rogierschouten @dex4er @bioball — please review this PR in the next few days. Be sure to explicitly select Approve or Request Changes in the GitHub UI so I know what's going on.

@typescript-bot typescript-bot moved this to Waiting for Code Reviews in Pull Request Status Board Jul 20, 2025
@typescript-bot typescript-bot moved this from Waiting for Code Reviews to Needs Maintainer Review in Pull Request Status Board Jul 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Needs Maintainer Review
Development

Successfully merging this pull request may close these issues.

2 participants