fix(ts): extend server type in Email provider from nodemailer

[justinforlenza]

Reasoning 💡

Adds additional typing for SMTP server config for tls & secure from nodemailer

Checklist 🧢

• Documentation
• Tests
• Ready to be merged

Affected issues 🎟

Fixes #807

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/nextauthjs/next-auth/BeFRU7tR7yRnt3GY6qcuZp6HQuig
✅ Preview: https://next-auth-git-fork-justinforlenza-patch-1-nextauthjs.vercel.app

[balazsorban44]

thanks, what about extending the whole option as described in #807 (comment)?

[justinforlenza]

thanks, what about extending the whole option as described in #807 (comment)?

Tbh idk why that just didn't even cross my mind, just made that change to just use the nodemailers built in interface

[balazsorban44]

Could you please make sure that you use npm@7 so the lock file doesn't get messed up? 😬 (you can verify by seeing lockFileVersion: 2 near the top)

[justinforlenza]

Could you please make sure that you use npm@7 so the lock file doesn't get messed up

Apoligies, I was trying out gitpod, and didnt realize it didnt have updated NPM

[balazsorban44]

Our TS tests were still failing. I added some suggestions to mitigate them, and I could seemingly also commit them to your fork, so I did. Hope it's OK. 😄

[codecov-commenter]

Codecov Report

Merging #2259 (3d984d8) into main (591aa7c) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main   #2259   +/-   ##
=====================================
  Coverage   9.97%   9.97%           
=====================================
  Files         82      82           
  Lines       1393    1393           
  Branches     393     393           
=====================================
  Hits         139     139           
  Misses      1028    1028           
  Partials     226     226           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 591aa7c...3d984d8. Read the comment docs.

[balazsorban44]

Thank you!