Ticket #4793 (closed defect (bug): invalid)

Opened 4 years ago

Last modified 4 years ago

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

Reported by: hendry Owned by: anonymous
Priority: normal Milestone:
Component: I18N Version: 2.2.2
Severity: normal Keywords: has-patch
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

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

Change History

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

I believe #4296 relates

  • Version set to 2.2.1

Setting version based on debian bug report

Nazgul4 years ago

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

-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   DD324 years ago

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

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   DD324 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.

  • Status changed from new to closed
  • Resolution set to invalid
  • Milestone 2.2.3 deleted
Note: See TracTickets for help on using tickets.