all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] [PATCH proxmox-perl-rs] notify context: fix 'default_sendmail_from' context method
@ 2023-10-02  7:40 Lukas Wagner
  2023-10-02 10:31 ` [pve-devel] applied: " Thomas Lamprecht
  0 siblings, 1 reply; 2+ messages in thread
From: Lukas Wagner @ 2023-10-02  7:40 UTC (permalink / raw)
  To: pve-devel

The name of the configuration option in datacenter.cfg is `email_from`
and not `mail_from`.

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>
---
Reported in our forum:
https://forum.proxmox.com/threads/mail-alerts-not-sent-with-datacenter-default.134305/

 pve-rs/src/notify_context.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pve-rs/src/notify_context.rs b/pve-rs/src/notify_context.rs
index 48623fd..3cf3e18 100644
--- a/pve-rs/src/notify_context.rs
+++ b/pve-rs/src/notify_context.rs
@@ -58,7 +58,7 @@ impl Context for PVEContext {
     fn default_sendmail_from(&self) -> String {
         let content = attempt_file_read("/etc/pve/datacenter.cfg");
         content
-            .and_then(|content| lookup_datacenter_config_key(&content, "mail_from"))
+            .and_then(|content| lookup_datacenter_config_key(&content, "email_from"))
             .unwrap_or_else(|| String::from("root"))
     }
 
-- 
2.39.2





^ permalink raw reply	[flat|nested] 2+ messages in thread

* [pve-devel] applied: [PATCH proxmox-perl-rs] notify context: fix 'default_sendmail_from' context method
  2023-10-02  7:40 [pve-devel] [PATCH proxmox-perl-rs] notify context: fix 'default_sendmail_from' context method Lukas Wagner
@ 2023-10-02 10:31 ` Thomas Lamprecht
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Lamprecht @ 2023-10-02 10:31 UTC (permalink / raw)
  To: Proxmox VE development discussion, Lukas Wagner

Am 02/10/2023 um 09:40 schrieb Lukas Wagner:
> The name of the configuration option in datacenter.cfg is `email_from`
> and not `mail_from`.
> 
> Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>
> ---
> Reported in our forum:
> https://forum.proxmox.com/threads/mail-alerts-not-sent-with-datacenter-default.134305/
> 
>  pve-rs/src/notify_context.rs | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
>

applied, thanks!




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-10-02 10:31 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-10-02  7:40 [pve-devel] [PATCH proxmox-perl-rs] notify context: fix 'default_sendmail_from' context method Lukas Wagner
2023-10-02 10:31 ` [pve-devel] applied: " Thomas Lamprecht

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal