🚨 [security] [js] Update nodemailer 6.7.7 → 6.9.9 (minor)

[depfu]

---

🚨 Your current dependencies have known security vulnerabilities 🚨

This dependency update fixes known security vulnerabilities. Please see the details below and assess their impact carefully. We recommend to merge and deploy this as soon as possible!

---

Here is everything you need to know about this update. Please take a good look at what changed and the test results before merging this pull request.

What changed?

✳️ nodemailer (6.7.7 → 6.9.9) · Repo · Changelog

Security Advisories 🚨

🚨 nodemailer ReDoS when trying to send a specially crafted email

Summary

A ReDoS vulnerability occurs when nodemailer tries to parse img files with the parameter attachDataUrls set, causing the stuck of event loop.
Another flaw was found when nodemailer tries to parse an attachments with a embedded file, causing the stuck of event loop.

Details

Regex: /^data:((?:[^;];)(?:[^,])),(.)$/

Path: compile -> getAttachments -> _processDataUrl

Regex: /(<img\b[^>]* src\s*=[\s"']*)(data:([^;]+);[^"'>\s]+)/

Path: _convertDataImages

PoC

https://gist.github.com/francoatmega/890dd5053375333e40c6fdbcc8c58df6
https://gist.github.com/francoatmega/9aab042b0b24968d7b7039818e8b2698

Impact

ReDoS causes the event loop to stuck a specially crafted evil email can cause this problem.

Release Notes

6.9.9

6.9.9 (2024-02-01)

Bug Fixes

• security: Fix issues described in GHSA-9h6g-pr28-7cqp. Do not use eternal matching pattern if only a few occurences are expected (dd8f5e8)
• tests: Use native node test runner, added code coverage support, removed grunt (#1604) (be45c1b)

6.9.8

6.9.8 (2023-12-30)

Bug Fixes

• punycode: do not use native punycode module (b4d0e0c)

6.9.7

6.9.7 (2023-10-22)

Bug Fixes

• customAuth: Do not require user and pass to be set for custom authentication schemes (fixes #1584) (41d482c)

6.9.6

6.9.6 (2023-10-09)

Bug Fixes

• inline: Use 'inline' as the default Content Dispostion value for embedded images (db32c93)
• tests: Removed Node v12 from test matrix as it is not compatible with the test framework anymore (7fe0a60)

6.9.5

6.9.5 (2023-09-06)

Bug Fixes

• license: Updated license year (da4744e)

6.9.4 (from changelog)

• Renamed SendinBlue to Brevo

6.9.3 (from changelog)

• Specified license identifier (was defined as MIT, actual value MIT-0)
• If SMTP server disconnects with a message, process it and include as part of the response error

6.9.2 (from changelog)

• Fix uncaught exception on invalid attachment content payload

6.9.1 (from changelog)

• Fix base64 encoding for emoji bytes in encoded words

6.9.0 (from changelog)

• Do not throw if failed to resolve IPv4 addresses
• Include EHLO extensions in the send response
• fix sendMail function: callback should be optional

6.8.0 (from changelog)

• Add DNS timeout (huksley)
• add dns.REFUSED (lucagianfelici)

6.7.8 (from changelog)

• Allow to use multiple Reply-To addresses

Does any of this look wrong? Please let us know.

Commits

See the full diff on Github. The new version differs by 44 commits:

• chore(master): release 6.9.9 [skip-ci] (#1606)
• fix(security): Fix issues described in GHSA-9h6g-pr28-7cqp. Do not use eternal matching pattern if only a few occurences are expected
• chore: do not use caret in version specifier
• fix(tests): Use native node test runner, added code coverage support, removed grunt (#1604)
• chore(master): release 6.9.8 [skip-ci] (#1605)
• chore: removed double file
• fix(punycode): do not use native punycode module
• Test new github notice syntax for README
• Updated stale github action
• chore: remove redundant AWS SDK for JavaScript v2 (#1593)
• chore: add mailcatch.app to well-known/services.json (#1586)
• chore(master): release 6.9.7 [skip-ci] (#1585)
• Merge branch 'master' of github.com:nodemailer/nodemailer
• fix(customAuth): Do not require user and pass to be set for custom authentication schemes (fixes #1584)
• update - `services.json` - add SES SMTP endpoints for the AP region (#1580)
• chore(master): release 6.9.6 [skip-ci] (#1578)
• fix(tests): Removed Node v12 from test matrix as it is not compatible with the test framework anymore
• fix(inline): Use 'inline' as the default Content Dispostion value for embedded images
• chore(master): release 6.9.5 [skip-ci] (#1573)
• fix(license): Updated license year
• Removed legacy files (#1571)
• Added release workflow to publish npm packages from Github
• Update services.json
• v6.9.4
• v6.9.4
• v6.9.3
• specified license identifier
• Update README.md
• Merge branch 'master' of github.com:nodemailer/nodemailer
• Process last message from server
• feat: added https://forwardemail.net to well-known services
• v6.9.2
• v6.9.1
• Updated examples
• fixed tests
• v6.9.0
• fix sendMail function: callback should be optional (#1490)
• Updated README
• v6.8.0
• Add DNS timeout (#1468)
• #1459 Add a failing test for attachments with null content (#1466)
• add dns.REFUSED (#1458)
• v6.7.8 (#1452)
• Updated readme

---

Depfu will automatically keep this PR conflict-free, as long as you don't add any commits to this branch yourself. You can also trigger a rebase manually by commenting with @depfu rebase.

All Depfu comment commands

@​depfu rebase

Rebases against your default branch and redoes this update

@​depfu recreate

Recreates this PR, overwriting any edits that you've made to it

@​depfu merge

Merges this PR once your tests are passing and conflicts are resolved

@​depfu cancel merge

Cancels automatic merging of this PR

@​depfu close

Closes this PR and deletes the branch

@​depfu reopen

Restores the branch and reopens this PR (if it's closed)

@​depfu pause

Ignores all future updates for this dependency and closes this PR

@​depfu pause [minor|major]

Ignores all future minor/major updates for this dependency and closes this PR

@​depfu resume

Future versions of this dependency will create PRs again (leaves this PR as is)