diff --git a/app/senders/smtp_sender.rb b/app/senders/smtp_sender.rb index 897282e4b..b762443be 100644 --- a/app/senders/smtp_sender.rb +++ b/app/senders/smtp_sender.rb @@ -142,10 +142,10 @@ def determine_mail_from_for_message(message) # If the domain has a valid custom return path configured, return # that. if message.domain.return_path_status == "OK" - return "#{message.server.token}@#{message.domain.return_path_domain}" + return "#{message.server.token}+#{message.token}@#{message.domain.return_path_domain}" end - "#{message.server.token}@#{Postal::Config.dns.return_path_domain}" + "#{message.server.token}+#{message.token}@#{Postal::Config.dns.return_path_domain}" end # Return the RCPT TO to use for the given message in this sending session diff --git a/spec/senders/smtp_sender_spec.rb b/spec/senders/smtp_sender_spec.rb index d61c532d8..5c20c04d7 100644 --- a/spec/senders/smtp_sender_spec.rb +++ b/spec/senders/smtp_sender_spec.rb @@ -261,7 +261,7 @@ sender.send_message(message) expect(sender.endpoints.last).to have_received(:send_message).with( kind_of(String), - "#{server.token}@#{domain.return_path_domain}", + "#{server.token}+#{message.token}@#{domain.return_path_domain}", ["john@example.com"] ) end @@ -272,7 +272,7 @@ sender.send_message(message) expect(sender.endpoints.last).to have_received(:send_message).with( kind_of(String), - "#{server.token}@#{Postal::Config.dns.return_path_domain}", + "#{server.token}+#{message.token}@#{Postal::Config.dns.return_path_domain}", ["john@example.com"] ) end @@ -306,7 +306,7 @@ it "adds the resent-sender header" do sender.send_message(message) expect(sender.endpoints.last).to have_received(:send_message).with( - "Resent-Sender: #{server.token}@#{Postal::Config.dns.return_path_domain}\r\n#{message.raw_message}", + "Resent-Sender: #{server.token}+#{message.token}@#{Postal::Config.dns.return_path_domain}\r\n#{message.raw_message}", kind_of(String), kind_of(Array) )