Skip to content
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

[FR] Notification Email plugin that supports TLS and STARTTLS protocol #5136

Open
2 tasks
tonhuisman opened this issue Oct 6, 2024 · 2 comments
Open
2 tasks

Comments

@tonhuisman
Copy link
Contributor

We already have N001 Email notifications plugin, and with #5135 we get support for sending email via an SSL encrypted connection.

Feature requests:

  • Add email notification support with the more modern TLS encryption protocol and STARTTLS (upgrade unencrypted connection to encrypted connection) support.
  • Send email notifications with attachments, like the data from the Cache controller, to recipient(s).

The features to support all this are available in this library, but as it adds quite substantial to the .bin size, it can not be included in all ESP32 builds, like the SSL encryption for N001, so it should be made available as a separate Notification plugin for user-defined Custom builds (and in MAX builds).

@TD-er
Copy link
Member

TD-er commented Oct 6, 2024

Other ideas, some of which are also implemented in devices like FritzBox modems:

  • Before upgrade email the settings as a ZIP file.
  • Allow to send some summary of collected data periodically (e.g. daily)
  • Allow to send some logs when something strange is happening (e.g. login from a different IP)
  • Send email with IP-settings when these change

These can benefit from the attachment feature, like sending some JSON with task data when the 'stats' are enabled.
And maybe even be able to generate some graphics, or send out captured pictures when we support camera sensors.

@TD-er
Copy link
Member

TD-er commented Oct 6, 2024

Some info about the differences between STARTTLS / SSL / TLS: https://mailtrap.io/blog/starttls-ssl-tls/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants