scalability: always reap children, RT#78270

[freeside.git] / FS / FS / Misc.pm
diff --git a/FS/FS/Misc.pm b/FS/FS/Misc.pm

index eedc736..66b3dda 100644 (file)
--- a/FS/FS/Misc.pm
+++ b/FS/FS/Misc.pm
@@ -156,7 +156,11 @@ sub send_email {
    
        unshift @mimeparts, { 
          'Type'        => ( $options{'content-type'} || 'text/plain' ),
-        'Data'        => $options{'body'},
+        'Charset'     => 'UTF-8',
+        'Data'        => ( $options{'content-type'} =~ /^text\//
+                             ? Encode::encode_utf8( $options{'body'} )
+                             : $options{'body'}
+                         ), 
          'Encoding'    => ( $options{'content-type'} ? '-SUGGEST' : '7bit' ),
          'Disposition' => 'inline',
        };
@@ -165,7 +169,10 @@ sub send_email {
      
        @mimeargs = (
          'Type'     => ( $options{'content-type'} || 'text/plain' ),
-        'Data'     => $options{'body'},
+        'Data'     => ( $options{'content-type'} =~ /^text\//
+                          ? Encode::encode_utf8( $options{'body'} )
+                          : $options{'body'}
+                      ),
          'Encoding' => ( $options{'content-type'} ? '-SUGGEST' : '7bit' ),
        );
  
@@ -254,13 +261,11 @@ sub send_email {
    }
  
    push @to, $options{bcc} if defined($options{bcc});
-  # make sure 
-  my @env_to = split(/\s*,\s*/, join(', ', @to));
-  # strip display-name from envelope addresses
-  foreach (@env_to) {
-    s/^\s*//;
-    s/\s*$//;
-    s/^(.*)\s*<(.*@.*)>$/$2/;
+  # fully unpack all addresses found in @to (including Bcc) to make the
+  # envelope list
+  my @env_to;
+  foreach my $dest (@to) {
+    push @env_to, map { $_->address } Email::Address->parse($dest);
    }
  
    local $@; # just in case
@@ -281,7 +286,7 @@ sub send_email {
    if ( $conf->exists('log_sent_mail') ) {
      my $cust_msg = FS::cust_msg->new({
          'env_from'  => $options{'from'},
-        'env_to'    => join(', ', @to),
+        'env_to'    => join(', ', @env_to),
          'header'    => $message->header_as_string,
          'body'      => $message->body_as_string,
          '_date'     => $time,
@@ -375,7 +380,7 @@ sub generate_email {
      'Type'        => 'text/plain',
      'Encoding'    => 'quoted-printable',
      #'Encoding'    => '7bit',
-    'Data'        => $data,
+    'Data'        => Encode::encode_utf8($data),
      'Disposition' => 'inline',
    );
  
@@ -396,7 +401,7 @@ sub generate_email {
                         '    </title>',
                         '  </head>',
                         '  <body bgcolor="#ffffff">',
-                       @html_data,
+                       ( map Encode::encode_utf8($_), @html_data ),
                         '  </body>',
                         '</html>',
                       ],
@@ -752,6 +757,8 @@ sub generate_ps {
  
    my $papersize = $conf->config('papersize') || 'letter';
  
+  local($SIG{CHLD}) = sub {};
+
    system('dvips', '-q', '-t', $papersize, "$file.dvi", '-o', "$file.ps" ) == 0
      or die "dvips failed";
  
@@ -809,6 +816,8 @@ sub generate_pdf {
    #system('dvipdf', "$file.dvi", "$file.pdf" );
    my $papersize = $conf->config('papersize') || 'letter';
  
+  local($SIG{CHLD}) = sub {};
+
    system(
      "dvips -q -f $sfile.dvi -t $papersize ".
      "| gs -q -dNOPAUSE -dBATCH -sDEVICE=pdfwrite -sOutputFile=$sfile.pdf ".
@@ -891,6 +900,7 @@ sub do_print {
                : $conf->config('lpr', $opt{'agentnum'} );
  
    my $outerr = '';
+  local($SIG{CHLD}) = sub {};
    run3 $lpr, $data, \$outerr, \$outerr;
    if ( $? ) {
      $outerr = ": $outerr" if length($outerr);
@@ -980,6 +990,8 @@ sub ocr_image {
    print $fh $logo_data;
    close $fh;
  
+  local($SIG{CHLD}) = sub {};
+
    run( [qw(ocroscript recognize), $filename], '>'=>"$filename.hocr" )
      or die "ocroscript recognize failed\n";