checking email address

[mwood_2k2]

Hi guys,

I recently got evolt.org's incoming mail script working, and I have added a mysql query to get email addresses from a database (I am planning on creating a mailing group) ... However, I am having trouble figuring out how I can set it up so I can check wether the incoming mail is valid (from a registered user) before handling it and sending it out to everyone on the list, both would require use of the same db, and I just can't think of an efficient way to do it... anyways heres my code, I hope u guys can help me, I'm an sql noob!

#!/usr/bin/php -q
<?php
$fd = fopen("php://stdin", "r");
$email = "";
while (!feof($fd)) {
    $email .= fread($fd, 1024);
}
fclose($fd);
$lines = explode("\n", $email);
$from = "";
$subject = "";
$headers = "";
$message = "";
$splittingheaders = true;
for ($i=0; $i<count($lines); $i++) {
    if ($splittingheaders) {
        $headers .= $lines[$i]."\n";
        if (preg_match("/^Subject: (.*)/", $lines[$i], $matches)) {
            $subject = $matches[1];
        }
        if (preg_match("/^From: (.*)/", $lines[$i], $matches)) {
            $from = $matches[1];
        }
    } else {
        $message .= $lines[$i]."\n";
    }
    if (trim($lines[$i])=="") {
        $splittingheaders = false;
    }
}

$link = mysql_connect("host", "user", "pass")
        or die("Could not connect: " . mysql_error());

mysql_select_db("database") or die("Could not select database");



$query = mysql_query("SELECT email from table");
$num = mysql_num_rows($query);

for ($i=0; $i<$num; $i++) {
$a = mysql_fetch_array($query);
$to = $a["email"]; 

mail("$to", "$subject", "$message", "From: [email][e-mail][/email]");
}
?>

Help me please guys !!

[tsinka]

Hi,

I hope I understood what you're trying to do.
In this case you would like to ensure that [e-mail] is a registered email adress before sending the mail to the other users ?

What about this:

$query = mysql_query("SELECT count(email) from table where email='[e-mail]'");
$num = mysql_num_rows($query);

if ($num) {
// valid
$query = mysql_query("SELECT email from table");
$num = mysql_num_rows($query);

for ($i=0; $i<$num; $i++) {
$a = mysql_fetch_array($query);
$to = $a["email"];

mail("$to", "$subject", "$message", "From: [e-mail]");
} else {
// invalid
do something
}

Regards,
Thomas

[tsinka]

Hi,

sorry, I forgot to insert the code with syntax highlighting 🙁

[mwood_2k2]

ahh, sorry, I guess I didn't explain myself too well...

what I mean is, I want to check the database to see if the email is from a valid user, so it basically looks through all the email addresses in the database and checks the address against those in its records to find a match, if it doesn't, then it does nothing...

its validation so I don't get people like emailing when they arent members and it going into the db and being mailed out etc, it needs to run the address of the mailer through the database, before calling on it to send out the mails to its members.

You understand?

So with your code, if I replace '[e-mail]' with $from in the query, which would be the mailers address .... that would work? Yes?

[tsinka]

I think that should work. The first query looks if the email address ($from) exists in the database (count > = 0 ?) and sends the email only if the address exists in the database.

[mwood_2k2]

I think it will ... no reason for it not to ... anyways, thanks very much! I wouldn't have been able to code it that efficiently, would have been longer winded coding lol.

Cheers again!

[mwood_2k2]

Agggh it doesn't work!

I get "parse error, unexpected T_ELSE in c:\program files\apache group\apache\htdocs\test.php on line 16" where line 16 is the line with else { on it .... any ideas??

if ($num) { 

$query = mysql_query("SELECT email from table"); 
$num = mysql_num_rows($query); 

for ($i=0; $i<$num; $i++) { 
$a = mysql_fetch_array($query); 
$to = $a["email"]; 

mail("$to", "$subject", "$message", "From: [email][e-mail][/email]"); 

} 

else { 

exit;

} 

As far as I can tell all statements have ;s and the proper }s and {s .. what can be wrong?

[srikanth1184]

Your for loop has a begin ({) but no end (}) !! I guess that would be the problem !!

[srikanth1184]

This must be a better approch!!

if ($num) 
{ 
$query = mysql_query("SELECT email from table"); 
$num = mysql_num_rows($query); 
while ($email = mysql_fetch_array($query) 
{ 
$to = $email["email"]; 
mail("$to", "$subject", "$message", "From: [email][e-mail][/email] <mailto:[e-mail]>"); 
}
} 
else
{ 
exit; 
}

[mwood_2k2]

Originally posted by srikanth1184
Your for loop has a begin ({) but no end (}) !! I guess that would be the problem !!

LOL yes, and I did try it with the end } and it still gave me the same error...

but anyways, now I'm getting a parse error on line 46, which is...

$num = mysql_num_rows($query);

so what is causing that?

cheers for help so far...

[srikanth1184]

Hi

Let us go one by one, Can you tell me what error you are getting when you place the end (}) for your while loop?

I don't have a crystal sphere to see what you have in your 46th line !!

[mwood_2k2]

I just showed what was in my 46th line ... lol

I'm getting the error when I email with a valid address to my list address, which pipes through to this script that were trying to fix... the line from the code ...

if ($num) 
{ 
$query = mysql_query("SELECT email from table"); 
$num = mysql_num_rows($query);  // this is line 46!
while ($email = mysql_fetch_array($query) 
{ 
$to = $email["email"]; 
mail("$to", "$subject", "$message", "From: [email][e-mail][/email] <mailto:[e-mail]>"); 
}
} 
else
{ 
exit; 
}

and I am only getting the failed delivery with that message when I send from a valid email address, the non valid ones get nada back....either that or its taking an EXTREMELY long time to return it.....I guess I'll find out soon enough!

So any suggestions? Sorry if I'm being a pain lol

edit: additionally I ran this code through my local apache with php ....

unexpected '{' in c:\program files\apache group\apache\htdocs\test.php on line 8

is what I got ... which is the following line ....

while ($email = mysql_fetch_array($query) 
{ // this is the supposed unexpected {

man I dunno wtf is going wrong ....

[srikanth1184]

If you are getting a parse error on line 46 the rest of the code should not work right !!

if you still working with the same code you have posted. take off the line 46 and close the while with ) braces

while ($email = mysql_fetch_array($query) ) // <--

[mwood_2k2]

Ok, now it works...

only, when I send mail from yahoo, its giving me this....

"--0-1237575416-1051831685=:22785
Content-Type: text/plain; charset=us-ascii

sdfsdf

---

Do you Yahoo!?
The New Yahoo! Search - Faster. Easier. Bingo.
--0-1237575416-1051831685=:22785
Content-Type: text/html; charset=us-ascii

<DIV>sdfsdf</DIV><p><hr SIZE=1>
Do you Yahoo!?<br>
<a href="http://us.rd.yahoo.com/search/mailsig/*[url]http://search.yahoo.com[/url]">The New
Yahoo! Search</a> - Faster. Easier. Bingo.
--0-1237575416-1051831685=:22785--
"

how can I stop these headers? my script doesnt seem to strip them out? Should I do a replace function for it?? Help! lol

PS: I recieve this type of thing only in my msn inbox...

Cheers