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