Make WordPress Core

Opened 17 years ago

Closed 17 years ago

#4296 closed defect (bug) (fixed)

wp_mail removes the Content-Type

Reported by: morty's profile morty Owned by: rob1n's profile rob1n
Milestone: 2.3 Priority: high
Severity: major Version: 2.2
Component: General Keywords: has-patch
Focuses: Cc:


wp_mail() removes the Content-Type. This breaks _all_ plugins not sending text-mails. Content-Type should be filtered just like the From headers.

Attachments (4)

PN_mailfix.php (2.6 KB) - added by morty 17 years ago.
Plugin which fixes the problem.…
wp_mail-rewrite.diff (6.3 KB) - added by rob1n 17 years ago.
wp-mail.fix.diff (3.3 KB) - added by rob1n 17 years ago.
4296.diff (1.2 KB) - added by Nazgul 17 years ago.

Download all attachments as: .zip

Change History (35)

17 years ago

Plugin which fixes the problem....

#1 @filosofo
17 years ago

You just need to use the PHPMailer API

$phpmailer->ContentType = "text/plain";

#2 @rob1n
17 years ago


#3 @rob1n
17 years ago

  • Owner changed from anonymous to rob1n
  • Status changed from new to assigned

#4 @rob1n
17 years ago

This will fix this ticket and add filters galore.

#5 @rob1n
17 years ago

  • Keywords has-patch added

#6 @rob1n
17 years ago

  • Keywords commit added

#7 follow-up: @mattyrob
17 years ago

I'm not 100% sure how phpmailer works but some of this code may be contradictory.

We are checking for Content-Type and setting it as text/plain if it is not set.

However, I have a plugin that passes some mail via wp_mail() that sets the Content-Type as text/html. With his diff the Content-Type will be left alone but the code still invokes $phpmailer->IsHTML( false ); which presumably tells phpmailer that the email isn't html when in fact it is!

Something like this might avoid this problem (if indeed it is still a problem after the patch.

if ( !isset( $content_type ) ) {
	$content_type = 'text/plain'; 
	$phpmailer->IsHTML( false );
} elseif ( isset( $content_type) && ($content_type == 'text/plain' ) {
	$phpmailer->IsHTML( false );
} elseif ( isset( $content_type) && ($content_type == 'text/html' ) {
	$phpmailer->IsHTML( true );

#8 in reply to: ↑ 7 @derjohng
17 years ago

... but the code still invokes $phpmailer->IsHTML( false ); which presumably tells phpmailer that the email isn't html when in fact it is!

I have the same problem. My "DJ-Email-Publish" plugin sends the post via wp_mail().

It works not well since 2.2, because wp_mail will be sent HTML mail ($phpmailer->IsHTML( false );).

#9 @rob1n
17 years ago

Okay, new logic that should fix that issue.

#10 @rob1n
17 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [5639]) wp_mail() rewrite that handles HTML mail. fixes #4296

#11 @ryan
17 years ago

I'm getting some reports of funkiness with mail when using the latest from the 2.2 branch.

I'm seeing weird stuff in my comment e-mails - escaped quotes and the
mime info in the msg body.

#12 @rob1n
17 years ago

Any examples?

I'll look through the code even so.

#13 @rob1n
17 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

#14 @rob1n
17 years ago

  • Keywords has-patch commit removed

#15 @ryan
17 years ago

Here's an example of mime being in the body. I removed personal info from the headers.

From: xxx
Date: June 4, 2007 10:48:39 AM MDT
To: xxx
Subject: [] Comment: "Foo is the new Bar"
Reply-To: xxx

MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Content-Type: text/plain; charset="UTF-8"

I asked the original reporters to drop a note on this ticket.

#16 @technosailor
17 years ago

Rob1n? ;)

#17 @rob1n
17 years ago

I've taken a look through the code, and upon visual inspection I can't find anything faulty, but I'm now going to be running specific debugs and tests.

See where that gets me.

17 years ago

#18 follow-up: @rob1n
17 years ago

Okay, I found some errors in my coding, but any extra headers I called wp_mail() with, nothing showed up in the body.

How did they call the wp_mail()?

#19 in reply to: ↑ 18 @technosailor
17 years ago

Replying to rob1n:

How did they call the wp_mail()?

I'm not sure I understand the question. Would be happy to provide more info from my end if I know what you are asking for. :)

#20 @rob1n
17 years ago

Like, what was the function call and the arguments?

I tried variations on the headers argument, but I still couldn't find the issue of headers in the body.

#21 @technosailor
17 years ago

As far as I can tell, it's every instance. For days, any notifications for moderation or new comments all have the weird headers that display into the message itself.

#22 @rob1n
17 years ago

Okay, I'm just going to yank this out of 2.2 and leave it in 2.3, where I will commence debugging. The only reason why I put this patch in 2.2 is because the reporter says it breaks HTML emails, but I guess it's plain text for now?

#23 @rob1n
17 years ago

(In [5668]) Revert [5639] for 2.2 branch, it's causing problems. see #4296

#24 @rob1n
17 years ago

  • Milestone changed from 2.2.1 to 2.3 (trunk)

Okay, bumping milestone to 2.3.

#25 @mattyrob
17 years ago

Just for information I was getting reports back from some WordPress 2.2 users of my plugin that Content-Type was being added as a header defined as text/plain and was also being added as text in the body of the email as text/html when my plugin was sending HTML emails.

If you want to see who my plugin constructs the emails see the publish (starts at line 327) and mail (starts at line 232) functions here:

#26 @mattyrob
17 years ago

Further update. I think the issue with my plugin was I was using Content-type with a small 't' whereas the wp_mail looks for Content-Type with a big 'T'. on not finding it the new header is added.

The problem here is that both may be valid depending on which document you read :-( For strict coding though it should bt 'T' - so my bad.

#27 @rob1n
17 years ago

I believe those headers are case-sensitive. There was a ticket that I committed in the last month or so that changed all mentions of "Content-type" to "Content-Type" in the WP source.

#28 @markjaquith
17 years ago

Like HTTP headers, MIME headers are not case sensitive. See section 1.2.2

WP should be comparing such headers in a case-insensitive way. If we're requiring a header to have a certain type of capitalization, that's a bug!

17 years ago

#29 @Nazgul
17 years ago

  • Keywords has-patch added

I've attached a patch which makes the relevant MIME header compares case-insensitive.

I've also made sure that that mime header is actually used, because the current code always adds a Content-Type: text/plain, whatever the input.

#30 @markjaquith
17 years ago

(In [5763]) Don't add MIME-Version header twice (PHPMailer already adds it). see #4296

#31 @markjaquith
17 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [5764]) Correctly grab Content-Type from old-school headers, skip blank headers. fixes #4296

Note: See TracTickets for help on using tickets.