Improved Mail Headers

[mattyrob]

I have written a plugin that I use on my site to improve the mail headers used by WordPress in the pluggable functions. Perhaps these changes could be added to the original code.

if ( !function_exists('wp_mail') ) :
function wp_mail($to, $subject, $message, $headers = '') {
	if( $headers == '' ) {
		$admin = get_userdata('1');
		$headers = "MIME-Version: 1.0\n" .
			"From: " .  $admin->display_name . " <" . $admin->user_email . ">\n" .
			"Return-Path: <" . $admin->user_email .">\n" .
			"Precedence: list\nList-Id: " . get_settings('blogname') . "\n" .
			"Content-Type: text/plain; charset=\"" . get_settings('blog_charset') . "\"\n";
	}
	return @mail($to, $subject, $message, $headers);	
}
endif;

[markjaquith]

Could you please explain the reasoning behind the changes? Thanks!

[mattyrob]

The changes are small and cosmetic hence the trivial severity rating.

The current wordpress installation sends emails from wordpress@<blog url>.

The changes send the emals from the admin (or first) user of the blog and also adds headers for a return address. I have also added a precendence line indication the email has been sent to a list (may not be appropriate to all) and also identifies the blog site in the list Id (this may help clients identify the email as other than spam)

[markjaquith]

Can't have it hardcoded to user #1. That user might not exist. wordpress@server was used because some servers were blocking e-mail that claimed to come from other domains.

[mattyrob]

I hadn't considered that user #1 might not exist. This also means that the Return Path is pointless as the wordpress@server address may not actually exist so replies would bounce.

Guess this can't be done but I'm still happy with my plugin :-)

[(none)]

Milestone 2.0.4 deleted