From mboxrd@z Thu Jan 1 00:00:00 1970
Return-Path: <pmg-devel-bounces@lists.proxmox.com>
Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68])
by lore.proxmox.com (Postfix) with ESMTPS id E6DB11FF15E
for <inbox@lore.proxmox.com>; Tue, 25 Feb 2025 11:30:18 +0100 (CET)
Received: from firstgate.proxmox.com (localhost [127.0.0.1])
by firstgate.proxmox.com (Proxmox) with ESMTP id 044701A390;
Tue, 25 Feb 2025 11:30:19 +0100 (CET)
Message-ID: <74515557-408e-4b39-ba33-adfe47424c8f@proxmox.com>
Date: Tue, 25 Feb 2025 11:29:45 +0100
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird Beta
To: Stoiko Ivanov <s.ivanov@proxmox.com>, pmg-devel@lists.proxmox.com
References: <20250224232451.399803-1-s.ivanov@proxmox.com>
<20250224232451.399803-2-s.ivanov@proxmox.com>
Content-Language: en-US
From: Dominik Csapak <d.csapak@proxmox.com>
In-Reply-To: <20250224232451.399803-2-s.ivanov@proxmox.com>
X-SPAM-LEVEL: Spam detection results: 0
AWL 0.022 Adjusted score from AWL reputation of From: address
BAYES_00 -1.9 Bayes spam probability is 0 to 1%
DMARC_MISSING 0.1 Missing DMARC policy
KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment
SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record
SPF_PASS -0.001 SPF: sender matches SPF record
Subject: Re: [pmg-devel] [PATCH pmg-api 1/7] config: add admin-mail-from key
X-BeenThere: pmg-devel@lists.proxmox.com
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Proxmox Mail Gateway development discussion
<pmg-devel.lists.proxmox.com>
List-Unsubscribe: <https://lists.proxmox.com/cgi-bin/mailman/options/pmg-devel>,
<mailto:pmg-devel-request@lists.proxmox.com?subject=unsubscribe>
List-Archive: <http://lists.proxmox.com/pipermail/pmg-devel/>
List-Post: <mailto:pmg-devel@lists.proxmox.com>
List-Help: <mailto:pmg-devel-request@lists.proxmox.com?subject=help>
List-Subscribe: <https://lists.proxmox.com/cgi-bin/mailman/listinfo/pmg-devel>,
<mailto:pmg-devel-request@lists.proxmox.com?subject=subscribe>
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Errors-To: pmg-devel-bounces@lists.proxmox.com
Sender: "pmg-devel" <pmg-devel-bounces@lists.proxmox.com>
high level:
wouldn't it make sense to squash this patch with the ones where it's actually used?
IMHO it only makes sense to split such things when the changes are rather
large, but that's not the case here.
On 2/25/25 00:24, Stoiko Ivanov wrote:
> To be used as From header address for mails generated by the PMG
> stack. Currently this is hardcoded to postmaster in a few places.
>
> While PMG generates mails as the mail-system and postmaster is a
> good choice for such mails, users sometimes wish to customize the
> e-mail shown e.g. in the backup-notification and the daily admin
> report.
>
> The set value will be used in the header only, following the similar
> setting 'mailfrom' in the 'spamquar' setting.
> I decided against reusing the existing setting, as it's specifically
> tied to a category.
>
> PMG will as keep sending most mails with an empty envelope-from or
> with 'postmaster'.
>
> This is a first step towards DKIM signing those mails.
>
> Users will have to opt-in to it and use a email with domain-part to
> get DKIM signing.
>
> I decided against changing the current behavior of using 'postmaster'
> without domain, as this makes postfix complete the address with
> '@$myorigin.$mydomain'[0], and I've seen quite a few installations
> which set those settings in their main.cf.in templates.
>
> [0] https://www.postfix.org/postconf.5.html#internal_mail_filter_classes
>
> Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
> ---
> src/PMG/Config.pm | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/src/PMG/Config.pm b/src/PMG/Config.pm
> index 3170241..c583cc6 100644
> --- a/src/PMG/Config.pm
> +++ b/src/PMG/Config.pm
> @@ -140,6 +140,11 @@ EODESC
> enum => [qw(header envelope)],
> default => 'envelope',
> },
> + 'admin-mail-from' => {
> + description => "Text for 'From' header in admin mails and bounces.",
> + type => 'string',
should we try to restrict this a bit?
> + default => 'postmaster',> + },
> };
> }
>
> @@ -159,6 +164,7 @@ sub options {
> dkim_sign_all_mail => { optional => 1 },
> dkim_selector => { optional => 1 },
> 'dkim-use-domain' => { optional => 1 },
> + 'admin-mail-from' => { optional => 1 },
> };
> }
>
_______________________________________________
pmg-devel mailing list
pmg-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pmg-devel