Ticket #4793 (closed defect: invalid)

Opened 11 months ago

Last modified 7 months ago

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

Reported by: hendry Assigned to: 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 (6.3 kB) - added by Nazgul on 08/21/07 20:53:58.

Change History

08/21/07 20:40:50 changed by westi

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

I believe #4296 relates

08/21/07 20:41:10 changed by westi

  • version set to 2.2.1.

Setting version based on debian bug report

08/21/07 20:53:58 changed by Nazgul

  • attachment 4793.diff added.

08/21/07 21:00:33 changed by Nazgul

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

12/09/07 12:18:45 changed by DD32

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

12/09/07 16:16:18 changed by darkdragon

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?

12/09/07 23:22:38 changed by DD32

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.

12/17/07 01:51:03 changed by thee17

  • status changed from new to closed.
  • resolution set to invalid.

12/17/07 09:35:11 changed by Nazgul

  • milestone deleted.