[fixed] email addresses with two periods return as invalid

[David Bott]

A user tried to register with an e-mail address such as...

user@kennedy.green-valley.az.us

VB did not seem to like the kennedy.green-valley.az.us domain and said the address was not valid.

Help please.

[tubedogg]

Yep, this is a bug...I'm going to move it to that forum.

It's not actually the long domain - it's the fact that there is more than one period after the @ sign.

[John]

Fixed

Replace line 248 of register.php with this:

if (!preg_match('/^[-!#$%&\'*+\\.\/0-9=?A-Z^_`{|}~]+@([-0-9A-Z]+\.)+([0-9A-Z]){2,4}$/i', $email)) {


John

[David Bott]

Thank you.

[David Bott]

UPDATE....From a member...

I'm having trouble reverifying the links below. When clicking on either, I get the message:


Parse error: parse error in
/dsk0/home/avsforum/www.avsforum.com/avs-vb/register.php on line 223


So it looks like the patch does not work.

[John]

David, try again. It appears the [php] code tag had lost some of the PHP code.

John

[David Bott]

Was it tested? 0 for 2 now....

Warning: Unexpected character in input: '\' (ASCII=92) state=1 in /dsk0/home/avsforum/www.avsforum.com/avs-vb/register.php on line 223

Warning: Unexpected character in input: '\' (ASCII=92) state=1 in /dsk0/home/avsforum/www.avsforum.com/avs-vb/register.php on line 223

Parse error: parse error
in /dsk0/home/avsforum/www.avsforum.com/avs-vb/register.php on line 223


Please test. :)

[t0ecutter]

I had the same thing happen to me after I replaced that line, just clicking on the Register link gave a bunch of PHP errors.

[DarkReaper]

For those of you with hacked vBs:

Find:


if (!preg_match("/^(.+)@[a-zA-Z0-9-]+\.[a-zA-Z0-9.]+$/si", $email)) {


And replace with John's code:


if (!preg_match('/^[-!#$%&'*+\\.\/0-9=?A-Z^_`{|}~]+@([-0-9A-Z]+\.)+([0-9A-Z]){2,4}$/i', $email)) {

[David Bott]

And for those of us without hacked VB? :)

[DarkReaper]

Then you find the line number he mentioned, or do a search for that code...by hacked vB I meant that the line numbers would've changed...

BTW, John, that code didn't work for me. I got a blank page when trying to do anything with register.php:

if (!preg_match('/^[-!#$%&'*+\\.\/0-9=?A-Z^_`{|}~]+@([-0-9A-Z]+\.)+([0-9A-Z]){2,4}$/i', $email)) {
eval("standarderror(\"".gettemplate("error_bademail")."\");");
}

[David Bott]

So then, you are saying the patch still does not fuction. Suggest removing the above posts so not to confuse others. (then I wall also need to remove mine.) :)