Fix for bug 117055: Emails were being truncated if they contained a line with nothing but a period on them. We now pass -i to

sendmail and its clones to tell it to ignore periods (since we close the pipe when we're done, rather than signalling it with
a period).  Has been tested with sendmail and postfix.
Patch by Dave Miller <justdave@syndicomm.com>
r= afranke, bugzilla@bkor.dhs.org, jake

8 files changed, 10 insertions, 10 deletions

diff --git a/Bugzilla/Token.pm b/Bugzilla/Token.pm
index f2c5fbde7..1cff2fac2 100644
--- a/Bugzilla/Token.pm
+++ b/Bugzilla/Token.pm
@@ -101,7 +101,7 @@ sub MailPasswordToken {
     my $emailsuffix = &::Param('emailsuffix');
     $token = &::url_quote($token);
 
-    open SENDMAIL, "|/usr/lib/sendmail -t";
+    open SENDMAIL, "|/usr/lib/sendmail -ti";
 
     print SENDMAIL qq|From: bugzilla-daemon
 To: $emailaddress$emailsuffix
@@ -144,7 +144,7 @@ sub Cancel {
     my $username = $realname ? $realname . " <" . $loginname . ">" : $loginname;
 
     # Notify the user via email about the cancellation.
-    open SENDMAIL, "|/usr/lib/sendmail -t";
+    open SENDMAIL, "|/usr/lib/sendmail -ti";
     print SENDMAIL qq|From: bugzilla-daemon
 To: $username
 Subject: "$tokentype" token cancelled
diff --git a/CGI.pl b/CGI.pl
index 9875b85dd..ab49fffcf 100644
--- a/CGI.pl
+++ b/CGI.pl
@@ -762,7 +762,7 @@ sub MailPassword {
                              "login" => $login,
                              "password" => $password});
 
-    open SENDMAIL, "|/usr/lib/sendmail -t";
+    open SENDMAIL, "|/usr/lib/sendmail -ti";
     print SENDMAIL $msg;
     close SENDMAIL;
 }
diff --git a/Token.pm b/Token.pm
index f2c5fbde7..1cff2fac2 100644
--- a/Token.pm
+++ b/Token.pm
@@ -101,7 +101,7 @@ sub MailPasswordToken {
     my $emailsuffix = &::Param('emailsuffix');
     $token = &::url_quote($token);
 
-    open SENDMAIL, "|/usr/lib/sendmail -t";
+    open SENDMAIL, "|/usr/lib/sendmail -ti";
 
     print SENDMAIL qq|From: bugzilla-daemon
 To: $emailaddress$emailsuffix
@@ -144,7 +144,7 @@ sub Cancel {
     my $username = $realname ? $realname . " <" . $loginname . ">" : $loginname;
 
     # Notify the user via email about the cancellation.
-    open SENDMAIL, "|/usr/lib/sendmail -t";
+    open SENDMAIL, "|/usr/lib/sendmail -ti";
     print SENDMAIL qq|From: bugzilla-daemon
 To: $username
 Subject: "$tokentype" token cancelled
diff --git a/globals.pl b/globals.pl
index 845e6ed9a..f88347f13 100644
--- a/globals.pl
+++ b/globals.pl
@@ -1433,7 +1433,7 @@ sub RemoveVotes {
             if (Param('sendmailnow')) {
                $sendmailparm = '';
             }
-            if (open(SENDMAIL, "|/usr/lib/sendmail $sendmailparm -t")) {
+            if (open(SENDMAIL, "|/usr/lib/sendmail $sendmailparm -ti")) {
                 my %substs;
 
                 $substs{"to"} = $name;
diff --git a/importxml.pl b/importxml.pl
index e3532c53a..584855e29 100755
--- a/importxml.pl
+++ b/importxml.pl
@@ -114,7 +114,7 @@ sub MailMessage {
   $header.= "Subject: $subject\n\n";
 
   open(SENDMAIL,
-    "|/usr/lib/sendmail -ODeliveryMode=background -t") ||
+    "|/usr/lib/sendmail -ODeliveryMode=background -ti") ||
       die "Can't open sendmail";
   print SENDMAIL $header . $message . "\n";
   close SENDMAIL;
diff --git a/move.pl b/move.pl
index 9647aee43..e82678d44 100755
--- a/move.pl
+++ b/move.pl
@@ -150,7 +150,7 @@ $msg .= "Subject: Moving bug(s) $buglist\n\n";
 $msg .= $xml . "\n";
 
 open(SENDMAIL,
-  "|/usr/lib/sendmail -ODeliveryMode=background -t") ||
+  "|/usr/lib/sendmail -ODeliveryMode=background -ti") ||
     die "Can't open sendmail";
 print SENDMAIL $msg;
 close SENDMAIL;
diff --git a/processmail b/processmail
index 844942728..4fcc424d2 100755
--- a/processmail
+++ b/processmail
@@ -771,7 +771,7 @@ sub NewProcessOnePerson ($$$$$$$$$$$$) {
     }
 
     if ($enableSendMail == 1) {
-    open(SENDMAIL, "|/usr/lib/sendmail $sendmailparam -t") ||
+    open(SENDMAIL, "|/usr/lib/sendmail $sendmailparam -ti") ||
       die "Can't open sendmail";
     
     print SENDMAIL trim($msg) . "\n";
diff --git a/whineatnews.pl b/whineatnews.pl
index ba258ddca..0b5d78eba 100755
--- a/whineatnews.pl
+++ b/whineatnews.pl
@@ -64,7 +64,7 @@ foreach my $email (sort (keys %bugs)) {
     }
 
     my $sendmailparam = Param('sendmailnow') ? '' : "-ODeliveryMode=deferred";
-    open SENDMAIL, "|/usr/lib/sendmail $sendmailparam -t"
+    open SENDMAIL, "|/usr/lib/sendmail $sendmailparam -ti"
         or die "Can't open sendmail";
     print SENDMAIL $msg;
     close SENDMAIL;