Fixed message headers encoding; improved recipient splitting; applied patch...

thomascube

2007-03-13 5a6ad209837a8bcca14d4f74541d8ac3ea760341

 program/steps/mail/sendmail.inc

@@ -100,21 +100,23 @@
    $image_name = substr($body,
                         $pos + strlen($searchstr),
                         $pos2 - ($pos + strlen($searchstr)));
    // sanitize image name so resulting attachment doesn't leave images dir
    $image_name = preg_replace('/[^a-zA-Z0-9_\.\-]/i','',$image_name);

    $body_post = substr($body, $pos2);

    if (! in_array($image_name, $included_images))
      {
      $body_post = substr($body, $pos2);

      // add the image to the MIME message
      $img_file = $INSTALL_PATH . '/' . $searchstr . $image_name;
      if(! $mime_message->addHTMLImage($img_file, 'image/gif', '', true, '_' . $image_name))
        {
        show_message("emoticonerror", 'error');
        }

      $body = $body_pre . 'cid:_' . $image_name . $body_post;
      array_push($included_images, $image_name);
      }

    $body = $body_pre . 'cid:_' . $image_name . $body_post;

    $last_img_pos = $pos2;
    }
@@ -149,8 +151,8 @@
$input_charset = $OUTPUT->get_charset();
$message_charset = isset($_POST['_charset']) ? $_POST['_charset'] : $input_charset;

$mailto_regexp = array('/[,;]\s*[\r\n]+/', '/[\r\n]+/', '/[,;]\s*$/m');
$mailto_replace = array(', ', ', ', '');
$mailto_regexp = array('/[,;]\s*[\r\n]+/', '/[\r\n]+/', '/[,;]\s*$/m', '/;/');
$mailto_replace = array(', ', ', ', '', ',');

// replace new lines and strip ending ', '
$mailto = preg_replace($mailto_regexp, $mailto_replace, get_input_value('_to', RCUBE_INPUT_POST, TRUE, $message_charset));
@@ -166,9 +168,9 @@
  $identity_arr['string'] = $from;

// compose headers array
$headers = array('Date' => date('D, j M Y G:i:s O'),
                 'From' => $identity_arr['string'],
                 'To'   => rcube_charset_convert($mailto, $input_charset, $message_charset));
$headers = array('Date' => date('D, j M Y H:i:s O'),
                 'From' => rcube_charset_convert($identity_arr['string'], $CHARSET, $message_charset),
                 'To'   => $mailto);

// additional recipients
if (!empty($_POST['_cc']))
@@ -295,12 +297,17 @@
                       'text_charset'  => $message_charset);

// compose message body and get headers
$msg_body = &$MAIL_MIME->get($message_param);
$msg_body = $MAIL_MIME->get($message_param);
// unset to save memory.
unset($MAIL_MIME->_parts);

$msg_subject = $headers['Subject'];

// encoding subject header with mb_encode provides better results with asian characters
if ($MBSTRING && function_exists("mb_encode_mimeheader"))
  $headers['Subject'] = mb_encode_mimeheader($headers['Subject'], $message_charset);
{
  mb_internal_encoding($message_charset);
  $mb_subject = mb_encode_mimeheader($headers['Subject'], $message_charset, 'Q');
  mb_internal_encoding($CHARSET);
}

// Begin SMTP Delivery Block 
if (!$savedraft) {
@@ -320,11 +327,12 @@
    $send_headers = $headers;
    unset($send_headers['Bcc']);
  
    // generate message headers
    $header_str = $MAIL_MIME->txtHeaders($send_headers);
    if (!empty($mb_subject))
      $send_headers['Subject'] = $mb_subject;
  
    // send message
    $sent = smtp_mail($from, $a_recipients, $header_str, $msg_body);
    $smtp_response = array();
    $sent = smtp_mail($from, $a_recipients, $MAIL_MIME->txtHeaders($send_headers), $msg_body, $smtp_response);
  
    // log error
    if (!$sent)
@@ -333,7 +341,7 @@
                        'type' => 'smtp',
                        'line' => __LINE__,
                        'file' => __FILE__,
                        'message' => "SMTP error: $SMTP_ERROR"), TRUE, FALSE);
                        'message' => "SMTP error: ".join("\n", $smtp_response)), TRUE, FALSE);
      }
    }
  
@@ -344,6 +352,9 @@
    $headers_enc = $MAIL_MIME->headers($headers);
    $headers_php = $MAIL_MIME->_headers;
    unset($headers_php['To'], $headers_php['Subject']);
    
    if (!empty($mb_subject))
      $headers_enc['Subject'] = $mb_subject;
    
    // reset stored headers and overwrite
    $MAIL_MIME->_headers = array();
@@ -446,12 +457,12 @@
  {
  if ($CONFIG['smtp_log'])
    {
    $log_entry = sprintf("[%s] User: %d on %s; Message for %s; Subject: %s\n",
    $log_entry = sprintf("[%s] User: %d on %s; Message for %s; %s\n",
                 date("d-M-Y H:i:s O", mktime()),
                 $_SESSION['user_id'],
                 $_SERVER['REMOTE_ADDR'],
                 $mailto,
                 $msg_subject);
                 !empty($smtp_response) ? join('; ', $smtp_response) : '');

    if ($fp = @fopen($CONFIG['log_dir'].'/sendmail', 'a'))
      {
@@ -462,7 +473,7 @@

  rcmail_compose_cleanup();
  rcube_iframe_response(sprintf("parent.$JS_OBJECT_NAME.sent_successfully('%s');",
                                rep_specialchars_output(rcube_label('messagesent'), 'js')));
                                JQ(rcube_label('messagesent'))));
  }