[ mailman-Patches-1906479 ] Add option for only munging Reply-To if it is not there

Patches item #1906479, was opened at 2008-03-03 20:45 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1906479&group_id=103 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: mail delivery Group: Mailman 2.1 Status: Open Resolution: None Priority: 5 Private: No Submitted By: Knut Auvor Grythe (auvor) Assigned to: Nobody/Anonymous (nobody) Summary: Add option for only munging Reply-To if it is not there Initial Comment: As I know a lot of you developers agree, Reply-To: header munging is not very nice. However, it appears to be a necessary evil sometimes. My biggest problem with Reply-To: header munging is that there is no way for a poster to direct replies off-list. Any attempt to do so will fail completely because MUAs obey the Reply-To: header, and the users don't notice. Ironically, these users are often the ones who made the munging necessary in the first place. However, I believe there is a solution to this problem. How about only adding a Reply-To: header only if one did not previously exist? This way, headers will be munged in the normal case, but if someone adds their own Reply-To: header for some reason, their value will be kept. Naturally, this behaviour should be optional, since it alters the behaviour of the lists. I have attached a patch which enables this functionality. The patch is against 2.1.9, but seems to apply on 2.1.10b1 without complaining. PS: This patch slightly changes the order in which munging is done, causing mlist.reply_to_address to be appended instead of prepended to the original Reply-To: header. I didn't consider this a problem, as it makes the behaviour if "This list" and "Explicit address" consistent. I also feel it made the code more straight-forward to read. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1906479&group_id=103
participants (1)
-
SourceForge.net