Use the address lists that have already been supplied to ensure we get the names as well

1 files changed, 5 insertions, 10 deletions

diff --git a/lib/action_mailbox/mail_ext/addresses.rb b/lib/action_mailbox/mail_ext/addresses.rb
index 94bb846d75..8059ce0027 100644
--- a/lib/action_mailbox/mail_ext/addresses.rb
+++ b/lib/action_mailbox/mail_ext/addresses.rb
@@ -1,26 +1,21 @@
 class Mail::Message
   def from_address
-    Mail::Address.new from.first
+    header[:from]&.address_list&.addresses&.first
   end
 
   def recipients_addresses
-    convert_to_addresses recipients
+    to_addresses + cc_addresses + bcc_addresses
   end
 
   def to_addresses
-    convert_to_addresses to
+    Array(header[:to]&.address_list&.addresses)
   end
 
   def cc_addresses
-    convert_to_addresses cc
+    Array(header[:cc]&.address_list&.addresses)
   end
 
   def bcc_addresses
-    convert_to_addresses bcc
+    Array(header[:bcc]&.address_list&.addresses)
   end
-
-  private
-    def convert_to_addresses(recipients)
-      Array(recipients).collect { |recipient| Mail::Address.new recipient }
-    end
 end