From 2231585c56b83d753db33722a9e60ee66375fabe Mon Sep 17 00:00:00 2001 From: Bob Vincent Date: Thu, 28 Apr 2011 20:32:48 -0400 Subject: [PATCH] Issue #209672 by dshaw, theborg, cburschka, Rob Loach, carlos8f, gdud, pillarsdotnet: Use site name in From: headers for system e-mails. --- includes/mail.inc | 10 ++++++++-- 1 files changed, 8 insertions(+), 2 deletions(-) diff --git a/includes/mail.inc b/includes/mail.inc index d2febed39686c9bf3f6f7a2bf99fa1377d09f4de..3da9b978e74da77e8750d741491cace7f9ab394d 100644 --- a/includes/mail.inc +++ b/includes/mail.inc @@ -119,15 +119,21 @@ function drupal_mail($module, $key, $to, $language, $params = array(), $from = N 'Content-Transfer-Encoding' => '8Bit', 'X-Mailer' => 'Drupal' ); - if ($default_from) { + if ($default_from && valid_email_address($default_from)) { // To prevent e-mail from looking like spam, the addresses in the Sender and // Return-Path headers should have a domain authorized to use the originating // SMTP server. - $headers['From'] = $headers['Sender'] = $headers['Return-Path'] = $default_from; + $headers['Sender'] = $headers['Return-Path'] = $headers['Errors-To'] = $default_from; } if ($from) { $headers['From'] = $from; } + else { + // Use site name in default From address. + $site_name = variable_get('site_name', 'Drupal'); + $site_name = str_replace('"', '', mime_header_encode($site_name)); + $headers['From'] = '"' . $site_name . '" <' . $default_from . '>'; + } $message['headers'] = $headers; // Build the e-mail (get subject and body, allow additional headers) by -- 1.7.4.1