sup

A curses threads-with-tags style email client

sup.git

git clone https://supmua.dev/git/sup/
commit d648aac104c71c464ee1ce1b859b9dcf1a26d0c8
parent 1d924353c43b751e282038780084b0deefbb7cc3
Author: William Morgan <wmorgan-sup@masanjin.net>
Date:   Fri, 27 Mar 2009 05:37:44 -0700

bugfix bugfix: preserve recipient_email email address

When responding to an email with a recipient_email field (i.e. an Envelope-To
header), look up the name in the account list, but keep the original email
address.

Diffstat:
M lib/sup/modes/reply-mode.rb | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/lib/sup/modes/reply-mode.rb b/lib/sup/modes/reply-mode.rb
@@ -64,10 +64,14 @@ EOS
     ## if we have a value from a hook, use it.
     from = if hook_reply_from
       hook_reply_from
-    ## otherwise, if the original email was addressed to a particular
-    ## address via an envelope-to or whatever, try and use that one.
+    ## otherwise, if the original email had an envelope-to header, try and use
+    ## it, and look up the corresponding name form the list of accounts.
+    ##
+    ## this is for the case where mail is received from a mailing lists (so the
+    ## To: is the list id itself). if the user subscribes via a particular
+    ## alias, we want to use that alias in the reply.
     elsif @m.recipient_email && (a = AccountManager.account_for(@m.recipient_email))
-      a
+      Person.new a.name, @m.recipient_email
     ## otherwise, try and find an account somewhere in the list of to's
     ## and cc's.
     elsif(b = (@m.to + @m.cc).find { |p| AccountManager.is_account? p })