WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

#4793 closed defect (bug) (invalid)

Setting the headers manually can break mail encoding in pluggable.php

Reported by: hendry Owned by:
Milestone: Priority: normal
Severity: normal Version: 2.2.2
Component: I18N Keywords: has-patch
Focuses: Cc:

Description

I have a user who says that the way pluggable.php is currently manually crafting the the mail headers can break for his setup.

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=436872
http://bugs.debian.org/cgi-bin/bugreport.cgi?msg=5;filename=wordpress.mail.patch;att=1;bug=436872

Attachments (1)

4793.diff (6.3 KB) - added by Nazgul 7 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 westi7 years ago

As far as I can tell this is fixed on trunk.

I believe #4296 relates

comment:2 westi7 years ago

  • Version set to 2.2.1

Setting version based on debian bug report

Nazgul7 years ago

comment:3 Nazgul7 years ago

  • Milestone changed from 2.4 (future) to 2.2.3
  • Version changed from 2.2.1 to 2.2.2

-1 to the "attached" patch as is, because it kills the "wp_mail_from" filter.

And I agree with westi that #4296 relates, so I think backporting those changes from trunk to 2.2.x is a better solution, if we want to fix this in the 2.2.x branch as well. I've attached a patch with those changes.

comment:4 DD326 years ago

Is this still anything of an issue? Or has it been delt with in another ticket?

comment:5 darkdragon6 years ago

Well, since most of the other 2.2.x tickets say that 2.2 is no longer maintained. Was this part of the issue fixed also, or is it a duplicate?

comment:6 DD326 years ago

Was this part of the issue fixed also

Well, Given it says its fixed on trunk(2.3) I doubt its still anything of an issue.

comment:7 thee176 years ago

  • Resolution set to invalid
  • Status changed from new to closed

comment:8 Nazgul6 years ago

  • Milestone 2.2.3 deleted
Note: See TracTickets for help on using tickets.