mail() header problem for html email

Question

I am trying to write a error reporting feature for a website in php. I cannot get the headers right so that the email will display as html.

Here is the code:

if( isset($_POST['submit']) )
{
    $browser = $_SERVER['HTTP_USER_AGENT'];
    $page = $_POST['page'];
    $email = $_POST['email'];
    $error = $_POST['error'];

    $message  =  "<html><body> \n";   
    $message .=  "Email: $email \n";
    $message .=  "Page: $page \n";
    $message .=  "OS/ Browser: $browser \n";
    $message .=  "Error: $error \n";
    $message .=  "</body></html> \n";

    $headers  = 'MIME-Version: 1.0' . '\r\n';
    $headers .= 'Content-type: text/html; charset="iso-8859-1"' . '\r\n';
    $headers .= 'From:  <[email protected]>' . '\r\n';
    $headers .= 'Reply-To: $email ' . '\r\n';
    $headers .= 'X-Priority: 1' . '\r\n';

    $subject  = "[ERROR REPORT] Page: " . $page;

    mail("[email protected]", $subject, $message, $headers );

    $mesg = "Thank you for your report!";

}

?>

drowe · Accepted Answer · 2009-05-12 20:16:00Z

2

Personally, I'm a fan of Pear Mail (http://pear.php.net/package/Mail) and Pear Mail_Mime (http://pear.php.net/package/Mail_Mime).

Sending an HTML e-mail (with a plain-text body, for clients that don't support HTML) is as simple as this:

include_once('Mail.php');
include_once('Mail/Mime.php');

$htmlBody = '<html><body><b>Hello World</b></body></html>';
$plainBody = 'Your client doesn\'t support HTML';
$em = Mail::factory('sendmail');
$headers = array('From'=>'[email protected]', 'To'=>'[email protected]', 'Subject'=>'Cool Email');
$mime = new Mail_Mime();
$mime->setTxtBody($plainBody);
$mime->setHtmlBody($htmlBody);
$message = $mime->get();
$headers = $mime->headers($headers);
$mail = $em->send('[email protected]', $headers, $message);

answered May 12, 2009 at 20:16

drowe

2,31818 silver badges14 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Peter O. · Accepted Answer · 2011-12-12 16:14:58Z

2

One problem I noticed is that you need to use double quotes instead of single quotes in the proper places.

Use: "\r\n"

Instead of: '\r\n'

edited Dec 12, 2011 at 16:14

Peter O.

33.1k14 gold badges86 silver badges97 bronze badges

answered Dec 16, 2010 at 17:10

Cookie Monster

211 bronze badge

Comments

Community · Accepted Answer · 2017-05-23 10:32:52Z

1

Basically repeating what I answered for another question, I'm all for rolling-your-own in most situations, but when it comes to mail I'd heartily recommend making it easier on yourself and using something like Swift Mailer or PHPMailer (in that order, for my money).

As a side-bonus (and assuming you specify reply-to, etc), you also have much less chance of being tagged as spam.

EDIT: Maybe it's just the example you've used, but there's no actual HTML in your message. Why not just use plain text? And yes, I'd use one of the classes I suggest for plain text too.

edited May 23, 2017 at 10:32

CommunityBot

11 silver badge

answered May 12, 2009 at 0:58

da5id

9,1569 gold badges42 silver badges53 bronze badges

2 Comments

Josh Curren Over a year ago

The example I used doesnt use html in it but in reality I do have html in there.

da5id Over a year ago

Suspected as much, but was just making the point - not so much for you even, more 'cos increasingly people seem to default to html email when often there's no need.

Collectives™ on Stack Overflow

mail() header problem for html email

3 Answers 3

Comments

Comments

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related