From bc724da289c6c26ebb6f59870cdecf9ccad07c6b Mon Sep 17 00:00:00 2001 From: Katrin Fischer Date: Sun, 7 Aug 2022 16:14:37 +0000 Subject: [PATCH] Bug 31281: Use correct reply-to email when sending overdue mails Overdue emails are either sent from the issuing or the home library of an item. We never use the patron's home library, so the reply-to address must explicitly be set in EnqueueLetter. To test: - Set up 2 branches (A and B) with different email addresses. - Set up an SMTP server for each to use - Set up an overdue notice trigger for the patron category you'll use First letter: 1 day delay, any notice - Check out an item with home branch B to a patron from A - Run the the script with: overdue_notices.pl -t --frombranch item-homebranch overdue_notices.pl -t --frombranch item-issuebranch - Confirm for each setting that the correct email headers have been used. You can see the reply-to address and to-address in the message queue: SELECT * FROM message_queue; Signed-off-by: Nason Bimbe Signed-off-by: Martin Renvoize Signed-off-by: Tomas Cohen Arazi (cherry picked from commit 18b2389de49b5c403f7bd68ef315b044a201eff5) Signed-off-by: Lucas Gass --- misc/cronjobs/overdue_notices.pl | 1 + 1 file changed, 1 insertion(+) diff --git a/misc/cronjobs/overdue_notices.pl b/misc/cronjobs/overdue_notices.pl index 9fdafe13b4..ba54e7cea4 100755 --- a/misc/cronjobs/overdue_notices.pl +++ b/misc/cronjobs/overdue_notices.pl @@ -803,6 +803,7 @@ END_SQL message_transport_type => $effective_mtt, from_address => $admin_email_address, to_address => join(',', @emails_to_use), + reply_address => $library->inbound_email_address, } ) unless $test_mode; # A print notice should be sent only once per overdue level.