Changeset 38058 for trunk/src/wp-includes/pluggable.php

Timestamp:

07/13/2016 06:03:52 PM (9 years ago)

Author:

boonebgorges

Message:

Mail: Improve handling of UTF-8 address headers.

Previously, wp_mail() implemented Reply-To as a generic header, using
PHPMailer's addCustomHeader(). As such, the email address portion of
the header was being incorrectly encoded when the name portion
contained UTF-8 characters. Switching to PHPMailer's more specific
addReplyTo() method fixes the issue.

For greater readability, the handling of all address-related headers
(To, CC, BCC, Reply-To) has been standardized.

Props szepe.viktor, iandunn, bpetty, stephenharris.
Fixes #21659.

File:

• trunk/src/wp-includes/pluggable.php (modified) (6 diffs)

trunk/src/wp-includes/pluggable.php

@@ -217 +217 @@
 
     // Headers
+    $cc = $bcc = $reply_to = array();
+
     if ( empty( $headers ) ) {
         $headers = array();
@@ -228 +230 @@
         }
         $headers = array();
-        $cc = array();
-        $bcc = array();
 
         // If it's actually got contents
@@ -292 +292 @@
                         $bcc = array_merge( (array) $bcc, explode( ',', $content ) );
                         break;
+                    case 'reply-to':
+                        $reply_to = array_merge( (array) $reply_to, explode( ',', $content ) );
+                        break;
                     default:
                         // Add it to our grand headers array
@@ -336 +339 @@
      * @param string $from_email Email address to send from.
      */
-    $phpmailer->From = apply_filters( 'wp_mail_from', $from_email );
+    $from_email = apply_filters( 'wp_mail_from', $from_email );
 
     /**
@@ -345 +348 @@
      * @param string $from_name Name associated with the "from" email address.
      */
-    $phpmailer->FromName = apply_filters( 'wp_mail_from_name', $from_name );
+    $from_name = apply_filters( 'wp_mail_from_name', $from_name );
+
+    $phpmailer->setFrom( $from_email, $from_name );
 
     // Set destination addresses
@@ -351 +356 @@
         $to = explode( ',', $to );
 
-    foreach ( (array) $to as $recipient ) {
-        try {
-            // Break $recipient into name and address parts if in the format "Foo <bar@baz.com>"
-            $recipient_name = '';
-            if ( preg_match( '/(.*)<(.+)>/', $recipient, $matches ) ) {
-                if ( count( $matches ) == 3 ) {
-                    $recipient_name = $matches[1];
-                    $recipient = $matches[2];
-                }
-            }
-            $phpmailer->AddAddress( $recipient, $recipient_name);
-        } catch ( phpmailerException $e ) {
-            continue;
-        }
-    }
-
     // Set mail's subject and body
     $phpmailer->Subject = $subject;
     $phpmailer->Body    = $message;
 
-    // Add any CC and BCC recipients
-    if ( !empty( $cc ) ) {
-        foreach ( (array) $cc as $recipient ) {
+    // Use appropriate methods for handling addresses, rather than treating them as generic headers
+    $address_headers = compact( 'to', 'cc', 'bcc', 'reply_to' );
+
+    foreach ( $address_headers as $address_header => $addresses ) {
+        if ( empty( $addresses ) ) {
+            continue;
+        }
+
+        foreach ( (array) $addresses as $address ) {
             try {
                 // Break $recipient into name and address parts if in the format "Foo <bar@baz.com>"
                 $recipient_name = '';
-                if ( preg_match( '/(.*)<(.+)>/', $recipient, $matches ) ) {
+
+                if ( preg_match( '/(.*)<(.+)>/', $address, $matches ) ) {
                     if ( count( $matches ) == 3 ) {
                         $recipient_name = $matches[1];
-                        $recipient = $matches[2];
+                        $address        = $matches[2];
                     }
                 }
-                $phpmailer->AddCc( $recipient, $recipient_name );
-            } catch ( phpmailerException $e ) {
-                continue;
-            }
-        }
-    }
-
-    if ( !empty( $bcc ) ) {
-        foreach ( (array) $bcc as $recipient) {
-            try {
-                // Break $recipient into name and address parts if in the format "Foo <bar@baz.com>"
-                $recipient_name = '';
-                if ( preg_match( '/(.*)<(.+)>/', $recipient, $matches ) ) {
-                    if ( count( $matches ) == 3 ) {
-                        $recipient_name = $matches[1];
-                        $recipient = $matches[2];
-                    }
+
+                switch ( $address_header ) {
+                    case 'to':
+                        $phpmailer->addAddress( $address, $recipient_name );
+                        break;
+                    case 'cc':
+                        $phpmailer->addCc( $address, $recipient_name );
+                        break;
+                    case 'bcc':
+                        $phpmailer->addBcc( $address, $recipient_name );
+                        break;
+                    case 'reply_to':
+                        $phpmailer->addReplyTo( $address, $recipient_name );
+                        break;
                 }
-                $phpmailer->AddBcc( $recipient, $recipient_name );
             } catch ( phpmailerException $e ) {
                 continue;