Make WordPress Core

Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#4793 closed defect (bug) (invalid)

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

Reported by: hendry's profile 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 17 years ago.

Download all attachments as: .zip

Change History (9)

#1 @westi
17 years ago

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

I believe #4296 relates

#2 @westi
17 years ago

  • Version set to 2.2.1

Setting version based on debian bug report

@Nazgul
17 years ago

#3 @Nazgul
17 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.

#4 @DD32
17 years ago

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

#5 @darkdragon
17 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?

#6 @DD32
17 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.

#7 @thee17
17 years ago

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

#8 @Nazgul
17 years ago

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