Make sure email replies to diary comments come from the right user

diff --git a/app/models/notifier.rb b/app/models/notifier.rb
index a498e4edf6cd88cb45c8e0be9bc8bb4f5d3bff2d..0539bdeb72fe3963bd7f52287427246713d035f0 100644 (file)
--- a/app/models/notifier.rb
+++ b/app/models/notifier.rb
@@ -107,7 +107,7 @@ class Notifier < ActionMailer::Base
                           :display_name => comment.user.display_name,
                           :title => "Re: #{comment.diary_entry.title}")
 
                           :display_name => comment.user.display_name,
                           :title => "Re: #{comment.diary_entry.title}")
 

-      mail :from => from_address(comment.user.display_name, "c", comment.id, comment.digest),
+      mail :from => from_address(comment.user.display_name, "c", comment.id, comment.digest, recipient.id),

            :to => recipient.email,
            :subject => I18n.t("notifier.diary_comment_notification.subject", :user => comment.user.display_name)
     end
            :to => recipient.email,
            :subject => I18n.t("notifier.diary_comment_notification.subject", :user => comment.user.display_name)
     end


@@ -180,9 +180,13 @@ class Notifier < ActionMailer::Base
     end
   end
 
     end
   end
 

-  def from_address(name, type, id, digest)
+  def from_address(name, type, id, digest, user_id = nil)

     if Object.const_defined?(:MESSAGES_DOMAIN) && domain = MESSAGES_DOMAIN
     if Object.const_defined?(:MESSAGES_DOMAIN) && domain = MESSAGES_DOMAIN

-      "#{name} <#{type}-#{id}-#{digest[0, 6]}@#{domain}>"
+      if user_id
+        "#{name} <#{type}-#{id}-#{user_id}-#{digest[0, 6]}@#{domain}>"
+      else
+        "#{name} <#{type}-#{id}-#{digest[0, 6]}@#{domain}>"
+      end

     else
       EMAIL_FROM
     end
     else
       EMAIL_FROM
     end

diff --git a/script/deliver-message b/script/deliver-message
index 04dcb881bc0699b3b948a9119b53d567565ecd6a..c15d365962e45124e6a7148c51f89dc6a22c6e88 100755 (executable)
--- a/script/deliver-message
+++ b/script/deliver-message
@@ -2,23 +2,25 @@
 
 require File.dirname(__FILE__) + "/../config/environment"
 
 
 require File.dirname(__FILE__) + "/../config/environment"
 

-exit 0 unless recipient = ARGV[0].match(/^([cm])-(\d+)-(.*)$/)
-
-if recipient[1] == "c"
-  comment = DiaryComment.find(recipient[2])
+if recipient = ARGV[0].match(/^c-(\d+)-(\d+)-(.*)$/)
+  comment = DiaryComment.find(recipient[1])

   digest = comment.digest
   digest = comment.digest

-  date = diary_comment.created_at
-  from = comment.diary_entry.user
+  date = comment.created_at
+  from = User.find(recipient[2])

   to = comment.user
   to = comment.user

-else
-  message = Message.find(recipient[2])
+  token = recipient[3]
+elsif recipient = ARGV[0].match(/^m-(\d+)-(.*)$/)
+  message = Message.find(recipient[1])

   digest = message.digest
   date = message.sent_on
   from = message.recipient
   to = message.sender
   digest = message.digest
   date = message.sent_on
   from = message.recipient
   to = message.sender

+  token = recipient[2]
+else
+  exit 0

 end
 
 end
 

-exit 0 unless recipient[3] == digest[0, 6]
+exit 0 unless token == digest[0, 6]

 exit 0 if date < 1.month.ago
 
 message.update_attribute(:message_read, true) if message
 exit 0 if date < 1.month.ago
 
 message.update_attribute(:message_read, true) if message