Need help with PHP Mail Script

[BlogMedia]

Hello everyone!

We have been working on this script for 2 days and can't seem to figure out whats wrong.

Its an online emailer, it uses the standard To:, From:, Subject and message but for some reason it wont send, so we check our server logs and get the following error...

A message that you sent contained a recipient address that was incorrectly
constructed:

Reply-To: missing or malformed local part (expected word or "<")

The message has not been delivered to any recipients.

------ This is a copy of your message, including all the headers. ------

To: [e-mail]

Email Recieved: [e-mail]
Email Subject: testing mail function

Message: ok here we go again see if this thing works

Here is the PHP code, and to ensure its not used for spam its in a protected area of our domain:

               $to_email = $to_email;    

               $msg = "$sitename\n\n";
               $msg .= "Email Recieved: $from_email\n";
               $msg .= "Email Subject: $email_subject\n\n";
               $msg .= "Message: $message\n\n";
               $mailheaders .= "Reply-To: '.$from_email.'\n\n";
               $mailheaders .= "From:  <$from_email>n"; 
               $mailheaders .= "Return-Path: <$from_email>n"; //Return bounced emails to. 
               $mailheaders .= "X-Priority: 1n";  //Message priority is set to HIGH 
               $mailheaders .= "X-MSMail-Priority: Highn"; //Message Priority for Exchange Servers 
               $mailheaders .= "Disposition-Notification-To: Support <$from_email>n"; //Read receipt 
               $mailheaders .= "X-Mailer: PHP(".$_SERVER['REMOTE_ADDR'].")n"; //IP address of who sent the mail
               $mailheaders .= "Reply-To: <$from_email>\n\n";
               $mailheaders .= "Content-Type: text/HTML; charset=ISO-8859-1\n"; 
               mail($to_email, $from_email, $email_subject, $msg, $mailheaders);
              

Now we assume it has something to do with the mailheaders, as the error message keeps mentioning this, however we have made several modifications and still it wont work...

Any help would be appreciated

BlogMedia

[NogDog]

Try getting rid of the quotes and angle brackets around the email addresses. Also, normally the header separator should be \r\n, not \n\n or just \n

//
               $mailheaders .= "Reply-To: $from_email\r\n";
               $mailheaders .= "From:  $from_email\r\n";
               $mailheaders .= "Return-Path: $from_email\r\n"; //Return bounced emails to.
               $mailheaders .= "X-Priority: 1\r\n";  //Message priority is set to HIGH
               $mailheaders .= "X-MSMail-Priority: High\r\n"; //Message Priority for Exchange Servers
               $mailheaders .= "Disposition-Notification-To: Support <$from_email>\r\n"; //Read receipt
               $mailheaders .= "X-Mailer: PHP(".$_SERVER['REMOTE_ADDR'].")\r\n"; //IP address of who sent the mail
               $mailheaders .= "Content-Type: text/HTML; charset=ISO-8859-1";

You also had the reply-to header in there twice, so I took out the second one.

[BlogMedia]

Thank you NogDog, for your reply but it still wont send, after testing the changes I see this in the server logs...

with same error...

A message that you sent contained a recipient address that was incorrectly
constructed:

Reply-To: missing or malformed local part (expected word or "<")

The message has not been delivered to any recipients.

------ This is a copy of your message, including all the headers. ------

To: [e-mail]
Subject: [e-mail]

Email Recieved: [e-mail]
Email Subject: Test Subject Line LALALALA

Message: Test MESSAGE LALALALA

Test Subject Line LALALALA

Here is the headers now...

$mailheaders = "Reply-To: $from_email\r\n"; 
        $mailheaders .= "From:  $from_email\r\n"; 
        $mailheaders .= "Return-Path: $from_email\r\n"; //Return bounced emails to. 
        $mailheaders .= "X-Priority: 1\r\n";  //Message priority is set to HIGH 
        $mailheaders .= "X-MSMail-Priority: High\r\n"; //Message Priority for Exchange Servers 
        $mailheaders .= "Disposition-Notification-To: Support $from_email\r\n"; //Read receipt 
        $mailheaders .= "X-Mailer: PHP(".$_SERVER['REMOTE_ADDR'].")\r\n"; //IP address of who sent the mail 
        $mailheaders .= "Content-Type: text/HTML; charset=ISO-8859-1"; 

Any other idea's?

BlogMedia

[BlogMedia]

Oops Double post, sorry !

Blog Media

[BlogMedia]

Does anyone know how to fix this script so it works?

BlogMedia:o

[NogDog]

Try commenting out the mail() command for now and print out the headers to see if anything is missing, mal-formed, or whatever, e.g.:

echo "<pre>$mailheaders</pre>\n";

[BlogMedia]

Oh thanks for the reply NogDog, as we are going crazy

Ok we did as requested and this is what was printed out...

Reply-To: [e-mail] From: [e-mail] Return-Path: [e-mail] X-Priority: 1 X-MSMail-Priority: High Disposition-Notification-To: [e-mail] X-Mailer: PHP(64.201.55.161) Content-Type: text/HTML; charset=ISO-8859-1

1

BlogMedia

[bradgrafelman]

Since some mailing agents improperly alter line endings, try replacing the "\r\n" sequences in your headers with just "\n" (only one). Uncomment the mail() call and see if it sends the mail that way.

[BlogMedia]

Hello I am back- Still the same thing....

Reply-To: missing or malformed local part (expected word or "<")

The message has not been delivered to any recipients.

------ This is a copy of your message, including all the headers. ------

My Mailheaders now;

       $mailheaders = "Reply-To: $from_email\n"; 
        $mailheaders .= "From:  $from_email\n"; 
        $mailheaders .= "Return-Path: $from_email\n"; //Return bounced emails to. 
        $mailheaders .= "X-Priority: 1\n";  //Message priority is set to HIGH 
        $mailheaders .= "X-MSMail-Priority: High\n"; //Message Priority for Exchange Servers 
        $mailheaders .= "Disposition-Notification-To: $from_email\n"; //Read receipt 
        $mailheaders .= "X-Mailer: PHP(".$_SERVER['REMOTE_ADDR'].")\n"; //IP address of who sent the mail 
        $mailheaders .= "Content-Type: text/HTML; charset=ISO-8859-1"; 
        mail($to_email, $from_email, $email_subject, $msg, $mailheaders);

Its so weird it wont go through ...

Blog Media

[bradgrafelman]

Try adding <>'s around all e-mail addresses that appear in headers, e.g. :

$mailheaders = "Reply-To: <$from_email>\r\n"; 
        $mailheaders .= "From: <$from_email>\r\n";

etc. Notice that I went back to use \r\n, as this is the proper way of writing e-mail headers, but if this doesn't work after adding the <>'s, try once again switching the line endings to simply "\n".

[BlogMedia]

OK did that and I get the following error message

Reply-To: missing or malformed local part (expected word or "<")

The message has not been delivered to any recipients.

------ This is a copy of your message, including all the headers. ------

Email subject from Mailer Deamon -- Mail failure - malformed recipient address

My Mail headers

        $mailheaders = "Reply-To: <$from_email>\r\n"; 
        $mailheaders .= "From: <$from_email>\r\n";
        $mailheaders .= "Return-Path: <$from_email>\r\n"; //Return bounced emails to. 
        $mailheaders .= "X-Priority: 1\n";  //Message priority is set to HIGH 
        $mailheaders .= "X-MSMail-Priority: High\n"; //Message Priority for Exchange Servers 
        $mailheaders .= "Disposition-Notification-To: <$from_email>\r\n"; //Read receipt 
        $mailheaders .= "X-Mailer: PHP(".$_SERVER['REMOTE_ADDR'].")\r\n"; //IP address of who sent the mail 
        $mailheaders .= "Content-Type: text/HTML; charset=ISO-8859-1"; 
        mail($to_email, $from_email, $email_subject, $msg, $mailheaders);

Blog Media

[bradgrafelman]

Just out of curiosity, can you echo out $to_email and show us what it looks like? Feel free to replace valid e-mail address with fake examples, but make sure to leave the basic structure intact for us to examine.

[BlogMedia]

I am not following you?

This is what I have now, what is it you want me to do

    if ($send != "no") {
	//$to_email = $to_email;   

//$to_email = [e-mail]; 
	$msg = "$sitename\n\n";
	$msg .= "Email Recieved: $from_email\n";
	$msg .= "Email Subject: $email_subject\n\n";
	$msg .= "Message: $message\n\n";
        $mailheaders = "Reply-To: <$from_email>\r\n"; 
        $mailheaders .= "From: <$from_email>\r\n";
        $mailheaders .= "Return-Path: <$from_email>\r\n"; //Return bounced emails to. 
        $mailheaders .= "X-Priority: 1\n";  //Message priority is set to HIGH 
        $mailheaders .= "X-MSMail-Priority: High\n"; //Message Priority for Exchange Servers 
        $mailheaders .= "Disposition-Notification-To: <$from_email>\r\n"; //Read receipt 
        $mailheaders .= "X-Mailer: PHP(".$_SERVER['REMOTE_ADDR'].")\r\n"; //IP address of who sent the mail 
        $mailheaders .= "Content-Type: text/HTML; charset=ISO-8859-1"; 
        mail($to_email, $from_email, $email_subject, $msg, $mailheaders);
//printf("<pre>%s</pre>\n", print_r($mailheaders)); 

Blog Media

[BlogMedia]

Ok I think I know what you mean so here is my email headers once again

	 $to_email = '[e-mail]';
	$msg = "$sitename\n\n";
	$msg .= "Email Recieved: $from_email\n";
	$msg .= "Email Subject: $email_subject\n\n";
	$msg .= "Message: $message\n\n";
        $mailheaders = "Reply-To: <$from_email>\r\n"; 
        $mailheaders .= "From: <$from_email>\r\n";
        $mailheaders .= "Return-Path: <$from_email>\r\n"; //Return bounced emails to. 
        $mailheaders .= "X-Priority: 1\n";  //Message priority is set to HIGH 
        $mailheaders .= "X-MSMail-Priority: High\n"; //Message Priority for Exchange Servers 
        $mailheaders .= "Disposition-Notification-To: <$from_email>\r\n"; //Read receipt 
        $mailheaders .= "X-Mailer: PHP(".$_SERVER['REMOTE_ADDR'].")\r\n"; //IP address of who sent the mail 
        $mailheaders .= "Content-Type: text/HTML; charset=ISO-8859-1"; 
        mail($to_email, $from_email, $email_subject, $msg, $mailheaders);

This is the error message

Reply-To: missing or malformed local part (expected word or "<")

The message has not been delivered to any recipients.

------ This is a copy of your message, including all the headers. ------

To: [e-mail]
Subject: [e-mail] <---- Again Should this be here?

Email Recieved: [e-mail]
Email Subject: subject line

Message: message body lalalala

Blog Media

[BlogMedia]

Ok I think I know what you mean so here is my email headers once again

	 $to_email = '[e-mail]';
	$msg = "$sitename\n\n";
	$msg .= "Email Recieved: $from_email\n";
	$msg .= "Email Subject: $email_subject\n\n";
	$msg .= "Message: $message\n\n";
        $mailheaders = "Reply-To: <$from_email>\r\n"; 
        $mailheaders .= "From: <$from_email>\r\n";
        $mailheaders .= "Return-Path: <$from_email>\r\n"; //Return bounced emails to. 
        $mailheaders .= "X-Priority: 1\n";  //Message priority is set to HIGH 
        $mailheaders .= "X-MSMail-Priority: High\n"; //Message Priority for Exchange Servers 
        $mailheaders .= "Disposition-Notification-To: <$from_email>\r\n"; //Read receipt 
        $mailheaders .= "X-Mailer: PHP(".$_SERVER['REMOTE_ADDR'].")\r\n"; //IP address of who sent the mail 
        $mailheaders .= "Content-Type: text/HTML; charset=ISO-8859-1"; 
        mail($to_email, $from_email, $email_subject, $msg, $mailheaders);

This is the error message

Reply-To: missing or malformed local part (expected word or "<")

The message has not been delivered to any recipients.

------ This is a copy of your message, including all the headers. ------

To: [e-mail]
Subject: [e-mail] <---- Again Should this be here?

Email Recieved: [e-mail]
Email Subject: subject line

Message: message body lalalala

Blog Media

[BlogMedia]

Ok I think I know what you mean so here is my email headers once again

	 $to_email = '[e-mail]';
	$msg = "$sitename\n\n";
	$msg .= "Email Recieved: $from_email\n";
	$msg .= "Email Subject: $email_subject\n\n";
	$msg .= "Message: $message\n\n";
        $mailheaders = "Reply-To: <$from_email>\r\n"; 
        $mailheaders .= "From: <$from_email>\r\n";
        $mailheaders .= "Return-Path: <$from_email>\r\n"; //Return bounced emails to. 
        $mailheaders .= "X-Priority: 1\n";  //Message priority is set to HIGH 
        $mailheaders .= "X-MSMail-Priority: High\n"; //Message Priority for Exchange Servers 
        $mailheaders .= "Disposition-Notification-To: <$from_email>\r\n"; //Read receipt 
        $mailheaders .= "X-Mailer: PHP(".$_SERVER['REMOTE_ADDR'].")\r\n"; //IP address of who sent the mail 
        $mailheaders .= "Content-Type: text/HTML; charset=ISO-8859-1"; 
        mail($to_email, $from_email, $email_subject, $msg, $mailheaders);

This is the error message

Reply-To: missing or malformed local part (expected word or "<")

The message has not been delivered to any recipients.

------ This is a copy of your message, including all the headers. ------

To: [e-mail]
Subject: [e-mail] <---- Again Should this be here?

Email Recieved: [e-mail]
Email Subject: subject line

Message: message body lalalala

Mail Delivery System 1:11 am Mail failure - malformed recipient address

Blog Media

[bradgrafelman]

OOPS - we've all been barking up the wrong tree here.

Your problem lies in the parameters you're passing to the mail() function. Take a look at the manual ([man]function.mail[/man]) to see what each parameter does.

[BlogMedia]

Dohhh, ok so how do we get this corrected now?

Blog Media

[bradgrafelman]

Well, you can see the correct order of parameters on the manual page I linked, but basically... you're trying to pass the "from" address as a separate parameter, and it should instead be defined in the headers (which you already have).

[BlogMedia]

Ok still a newb here with PHP, what is wrong with this...?