-
-
Notifications
You must be signed in to change notification settings - Fork 18
Open
Description
mailauth@4.12.0 depends on undici@7.16.0 (since undici@7.13.0 it requires global File class available which is only natively in Node v20.0.0+). right now package.json has set node v18, which is incorrect. if you use mailauth on node v18, you will get the following error:
ReferenceError: File is not defined
› Object.<anonymous> (node_modules/.pnpm/undici@7.16.0/node_modules/undici/lib/web/webidl/index.js:531:48)
› Object.<anonymous> (node_modules/.pnpm/undici@7.16.0/node_modules/undici/lib/web/fetch/util.js:12:20)
› Object.<anonymous> (node_modules/.pnpm/undici@7.16.0/node_modules/undici/lib/web/fetch/headers.js:11:5)
› Object.<anonymous> (node_modules/.pnpm/undici@7.16.0/node_modules/undici/lib/web/fetch/response.js:3:90)
› Object.<anonymous> (node_modules/.pnpm/undici@7.16.0/node_modules/undici/lib/web/fetch/index.js:12:5)
› Object.<anonymous> (node_modules/.pnpm/undici@7.16.0/node_modules/undici/index.js:119:19)
› Object.<anonymous> (node_modules/.pnpm/mailauth@4.12.0/node_modules/mailauth/lib/bimi/index.js:15:36)
› Object.<anonymous> (node_modules/.pnpm/mailauth@4.12.0/node_modules/mailauth/lib/mailauth.js:8:48)
› Object.<anonymous> (helpers/process-email.js:25:26)
› Object.<anonymous> (test/api/v1.js:51:22)see nodejs/undici#4374 - a simple fix to downgrade to a previous version of undici might be nice. there are some major performance issues in node v20, v22, etc which I documented below, so I cannot upgrade to node v20+ yet.
nodejs/node#60719 – this fix along might not even make it into node v20 EOL, not sure.
Metadata
Metadata
Assignees
Labels
No labels