Allow for unsecure SMTP settings for local unencrypted smtp relays

[clivinghouse]

This update provides the flexibility for users to configure SMTP for local instances where encryption is not required. For environments using an unencrypted SMTP relay or similar local instances that are not open to the public, the end user can now choose to disable encryption. This ensures flexibility in the configuration while still maintaining the option for secure SMTP connections in other environments.

From NodeMailer

secure – if true the connection will use TLS when connecting to server. If false (the default) then TLS is used if server supports the STARTTLS extension. In most cases set this value to true if you are connecting to port 465. For port 587 or 25 keep it false.

ignoreTLS – if this is true and secure is false then TLS is not used even if the server supports STARTTLS extension.

[github-actions]

Welcome!

Hello there, congrats on your first PR! We're excited to have you contributing to this project.
By submitting your Pull Request, you acknowledge that you agree with the terms of our Contributor License Agreement.

Generated by 🚫 dangerJS against 7f546bc

[greptile-apps]

PR Summary

This PR adds support for unsecured SMTP connections by introducing the EMAIL_SMTP_NO_TLS environment variable for local development environments.

• Added EMAIL_SMTP_NO_TLS environment variable with proper validation in environment-variables.ts
• Modified email.module-factory.ts to check for the noTLS flag and set both secure: false and ignoreTLS: true when enabled
• Refactored SMTP configuration to use an options object for better organization
• Improved error handling for missing SMTP host/port configuration
• Implemented according to NodeMailer documentation for completely disabling TLS encryption

_{💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!}

_{2 file(s) reviewed, 2 comment(s)
Edit PR Review Bot Settings | Greptile}