From: Lukas Wagner <l.wagner@proxmox.com>
To: Proxmox VE development discussion <pve-devel@lists.proxmox.com>,
Gabriel Goller <g.goller@proxmox.com>
Subject: Re: [pve-devel] [PATCH common] fix #4162: added `Auto-Submitted` header to email body
Date: Mon, 28 Aug 2023 11:20:04 +0200 [thread overview]
Message-ID: <51a36c9a-ffa8-4916-b49b-28c6ae13ce14@proxmox.com> (raw)
In-Reply-To: <20230828085134.39413-1-g.goller@proxmox.com>
On 8/28/23 10:51, Gabriel Goller wrote:
> diff --git a/src/PVE/Tools.pm b/src/PVE/Tools.pm
> index 9ffac12..5b5d72c 100644
> --- a/src/PVE/Tools.pm
> +++ b/src/PVE/Tools.pm
> @@ -1603,6 +1603,7 @@ sub sendmail {
>
> if (defined($html)) {
> print $mail "Content-Type: text/html;\n";
> + print $mail "Auto-Submitted: auto-generated;\n";
> print $mail "\tcharset=\"UTF-8\"\n";
> print $mail "Content-Transfer-Encoding: 8bit\n";
> print $mail "\n";
Just as a side-note: Since my notification patches have been merged a
week or two ago, PVE does not actually use PVE::Tools::sendmail any
more. Instead, it uses the equivalent Rust implementation of that
function from `proxmox_sys::mail::sendmail`.
It would make sense to include this change there as well, then we have
all products covered (PMG uses the Perl impl, PBS and PVE use the Rust
impl).
Apart from that: From briefly skimming through the RFC the change makes
sense, so consider this:
Reviewed-by: Lukas Wagner <l.wagner@proxmox.com>
--
- Lukas
next prev parent reply other threads:[~2023-08-28 9:20 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-28 8:51 Gabriel Goller
2023-08-28 9:20 ` Lukas Wagner [this message]
2023-08-28 9:30 ` Gabriel Goller
2023-08-28 9:33 ` Lukas Wagner
2023-09-19 7:36 ` Gabriel Goller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51a36c9a-ffa8-4916-b49b-28c6ae13ce14@proxmox.com \
--to=l.wagner@proxmox.com \
--cc=g.goller@proxmox.com \
--cc=pve-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox