Updated CookHeaders to always add the poster's From: address to Reply-To:

when applying from_is_list transformations.

1 files changed, 2 insertions, 8 deletions

diff --git a/Mailman/Handlers/CookHeaders.py b/Mailman/Handlers/CookHeaders.py
index 4fbfa311..02d09c0d 100755
--- a/Mailman/Handlers/CookHeaders.py
+++ b/Mailman/Handlers/CookHeaders.py
@@ -166,17 +166,11 @@ def process(mlist, msg, msgdata):
         # cases we'll zap the existing field because RFC 2822 says max one is
         # allowed.
         if not mlist.first_strip_reply_to:
-            # If we Munged the From:, add it to Reply-To: if we're not
-            # stripping it.
-            #MAS ? Should we add it anyway?
-            if o_from:
-                add(o_from)
             orig = msg.get_all('reply-to', [])
             for pair in getaddresses(orig):
                 add(pair)
-        # We also need to put the old From: in Reply-To: if reply_goes_to_list
-        # is to the poster even if we're stripping Reply-To:
-        if mlist.reply_goes_to_list == 0 and o_from:
+        # We also need to put the old From: in Reply-To: in all cases.
+        if o_from:
             add(o_from)
         # Set Reply-To: header to point back to this list.  Add this last
         # because some folks think that some MUAs make it easier to delete