[wp-trac] [WordPress Trac] #14888: PHPMailer class uses wrong/no sender for mail envelope

WordPress Trac wp-trac at lists.automattic.com
Wed Jan 4 17:53:53 UTC 2012


#14888: PHPMailer class uses wrong/no sender for mail envelope
-------------------------------+-----------------------------
 Reporter:  gkusardi           |       Owner:
     Type:  defect (bug)       |      Status:  new
 Priority:  normal             |   Milestone:  Future Release
Component:  Mail               |     Version:  3.0
 Severity:  normal             |  Resolution:
 Keywords:  reporter-feedback  |
-------------------------------+-----------------------------
Changes (by SergeyBiryukov):

 * keywords:  reporter-feedback close => reporter-feedback


Comment:

 Replying to [comment:7 cnorris23]:
 > This seems to have been addressed at least in PHPMailer 5.2 around line
 766, code below:
 >
 > {{{
 > $smtp_from = ($this->Sender == '') ? $this->From : $this->Sender;
 > }}}

 That line exists since PHPMailer was introduced in #3862: [[BR]]
 http://core.trac.wordpress.org/browser/tags/2.2/wp-includes/class-
 phpmailer.php#L467 [[BR]]
 It doesn't seem to address the issue reported here though.

 If we should remove "Return-Path", then [attachment:wp-phpmailer.diff]
 probably makes sense, but I don't get what the first change (removing
 `$smtp_from`) is there for. Looks like only the second one is needed.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/14888#comment:9>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list