[pve-devel] [PATCH v3 many 0/7] notifications: add SMTP endpoint

Gilberto Ferreira gilberto.nunes32 at gmail.com
Sat Sep 23 14:52:40 CEST 2023


Hi there

Today I saw an update here in PVE 8 that put some panels in order to
configure this endpoint, I believe.
But after installing this update, I could not find where the panels are.
Any hint? Thanks






Em seg., 18 de set. de 2023 às 08:15, Lukas Wagner <l.wagner at proxmox.com>
escreveu:

> This patch series adds support for a new notification endpoint type,
> smtp. As the name suggests, this new endpoint allows PVE to talk
> to SMTP server directly, without using the system's MTA (postfix).
>
> On the Rust side, these patches add a new dependency to the `lettre`
> crate for SMTP communication. This crate was chosen as it is:
>   - by far the most popular mailing crate for Rust
>   - well maintained
>   - has reasonable dependencies
>   - has async support, enabling us to asyncify the proxmox-notify
>     crate at some point, if needed
>
> These patches are based on top of the `system mail` patch series [1].
> From [1], we need the patches for `proxmox` and `proxmox-perl-rs`.
> If `pve-docs` from [1] is not applied first, there might be a small
> conflict,
> however that one should be trivial to resolve.
>
> Tested against
>   - the gmail SMTP server
>   - the posteo SMTP server
>   - our own webmail SMTP server
>
> Changes since v2:
>   - Rebased proxmox-widget-toolkit onto the latest master to avoid
>     any conflicts.
>
> Changes since v1:
>   - Rebased on top of [1]
>   - Added a mechanism for mails forwarded by `proxmox-mail-forward`
>     These are forwarded inline as "message/rfc822" to avoid having
>     to rewrite mail headers (otherwise, some SMTP relays might reject the
>     mail, because the `From` header of the forwarded mail does not match
> the
>     mail account)
>
> [1] https://lists.proxmox.com/pipermail/pve-devel/2023-August/058956.html
>
>
>
> proxmox:
>
> Lukas Wagner (3):
>   notify: fix typo in doc comments
>   notify: add 'smtp' endpoint
>   notify: add api for smtp endpoints
>
>  Cargo.toml                                  |   1 +
>  proxmox-notify/Cargo.toml                   |   4 +-
>  proxmox-notify/src/api/mod.rs               |  48 +++
>  proxmox-notify/src/api/smtp.rs              | 373 ++++++++++++++++++++
>  proxmox-notify/src/config.rs                |  23 ++
>  proxmox-notify/src/endpoints/common/mail.rs |  24 ++
>  proxmox-notify/src/endpoints/common/mod.rs  |   2 +
>  proxmox-notify/src/endpoints/mod.rs         |   4 +
>  proxmox-notify/src/endpoints/sendmail.rs    |  22 +-
>  proxmox-notify/src/endpoints/smtp.rs        | 260 ++++++++++++++
>  proxmox-notify/src/lib.rs                   |  19 +-
>  11 files changed, 761 insertions(+), 19 deletions(-)
>  create mode 100644 proxmox-notify/src/api/smtp.rs
>  create mode 100644 proxmox-notify/src/endpoints/common/mail.rs
>  create mode 100644 proxmox-notify/src/endpoints/common/mod.rs
>  create mode 100644 proxmox-notify/src/endpoints/smtp.rs
>
>
> proxmox-perl-rs:
>
> Lukas Wagner (1):
>   notify: add bindings for smtp API calls
>
>  common/src/notify.rs | 110 +++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 110 insertions(+)
>
>
> pve-manager:
>
> Lukas Wagner (1):
>   notify: add API routes for smtp endpoints
>
>  PVE/API2/Cluster/Notifications.pm | 337 ++++++++++++++++++++++++++++++
>  1 file changed, 337 insertions(+)
>
>
> proxmox-widget-toolkit:
>
> Lukas Wagner (1):
>   panel: notification: add gui for SMTP endpoints
>
>  src/Makefile                     |   2 +
>  src/Schema.js                    |   5 +
>  src/panel/EmailRecipientPanel.js |  89 ++++++++++++++
>  src/panel/SendmailEditPanel.js   |  67 ++---------
>  src/panel/SmtpEditPanel.js       | 192 +++++++++++++++++++++++++++++++
>  5 files changed, 296 insertions(+), 59 deletions(-)
>  create mode 100644 src/panel/EmailRecipientPanel.js
>  create mode 100644 src/panel/SmtpEditPanel.js
>
>
> pve-docs:
>
> Lukas Wagner (1):
>   notifications: document SMTP endpoints
>
>  notifications.adoc | 28 ++++++++++++++++++++++++++++
>  1 file changed, 28 insertions(+)
>
>
> Summary over all repositories:
>   19 files changed, 1532 insertions(+), 78 deletions(-)
>
> --
> murpp v0.4.0
>
>
>
> _______________________________________________
> pve-devel mailing list
> pve-devel at lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
>
>



More information about the pve-devel mailing list