[Patchew-devel] [PATCH] email: tweak rules for including the original sender as a recipient

Paolo Bonzini posted 1 patch 1 week ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/patchew next tags/patchew/20190514100126.9830-1-pbonzini@redhat.com
mods/email.py | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)

[Patchew-devel] [PATCH] email: tweak rules for including the original sender as a recipient

Posted by Paolo Bonzini 1 week ago
If the To field is empty, always include the sender as a recipient.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 mods/email.py | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/mods/email.py b/mods/email.py
index 4d71c9f..3ac1265 100644
--- a/mods/email.py
+++ b/mods/email.py
@@ -63,7 +63,7 @@ Email information is configured in "INI" style:
                                       desc="Whether this event is enabled",
                                       default=True),
                         BooleanSchema("reply_to_all", "Reply to all",
-                                      desc='Whether to "reply to all" if the event has an associated email message',
+                                      desc='If set, Cc all the receipients of the email message associated to the event. Also, if set the original sender of the email message will be a recipient even if the "to" field is nonempty',
                                       default=False),
                         BooleanSchema("in_reply_to", "Set In-Reply-To",
                                       desc='Whether to set In-Reply-To to the message id, if the event has an associated email message',
@@ -228,9 +228,11 @@ Email information is configured in "INI" style:
                 cc = [x.strip() for x in Template(nt["cc"]).render(ctx).strip().split()]
             except EmailCancelled:
                 continue
-            if nt["reply_to_all"] and mo:
-                to += [mo.get_sender_addr()]
-                cc += [x[1] for x in mo.recipients]
+            if mo:
+                if nt["reply_to_all"] or not len(to):
+                    to += [mo.get_sender_addr()]
+                if nt["reply_to_all"]:
+                    cc += [x[1] for x in mo.recipients]
             if mo and nt["in_reply_to"]:
                 headers["In-Reply-To"] = "<%s>" % mo.message_id
             if mo and nt["set_reply_to"]:
-- 
2.21.0

_______________________________________________
Patchew-devel mailing list
Patchew-devel@redhat.com
https://www.redhat.com/mailman/listinfo/patchew-devel