Email validation my 1st MOD on MWG!

[kcmajor36]

Okay first replace your register.php with:

 

You're unable to view this code.

Viewing code within this forum requires registration, you can register here for free.

 

Then name this one activation_cron.php

 

You're unable to view this code.

Viewing code within this forum requires registration, you can register here for free.

 

Then add this to you SQL:

[mysql]CREATE TABLE `confirm` (

`user` INT( 11 ) NOT NULL ,

`code` BIGINT( 40 ) NOT NULL

) ENGINE = MYISAM ;

 

ALTER TABLE users ADD `confirmed` int(11) NOT NULL ;

UPDATE users SET confirmed=1[/mysql]

Then last but not least, add this to your cron jobs as a two hour cron:

0 */2 * * * * curl http://yourgameurl.com/activation_cron.php?code=713518314

[Danny696]

One word: Insecure

[kcmajor36]

One word: Insecure

If theres a problem then post a fix on here!

[Brad]

I like it but not secure kyle good for your first mod

+1

[kcmajor36]

If you guys could help me secure it that would be great!

[Danny696]

Problem

$IP = ($_SERVER['HTTP_X_FORWARDED_FOR'])

? $_SERVER['HTTP_X_FORWARDED_FOR']

: $_SERVER['REMOTE_ADDR'];

Fix

$IP = $_SERVER['REMOTE_ADDR];

[kcmajor36]

Fixed! ;)

[Danny696]

Woops, should be

$IP = $_SERVER['REMOTE_ADDR'];

[Danny696]

The first file will error ;)

[kcmajor36]

Fixed errors!

[Danny696]

Fixed errors, unless your on php 5.3.1 (ties in with next part ;))

and still insecure

$IP = ($_SERVER['HTTP_X_FORWARDED_FOR'])

? $_SERVER['HTTP_X_FORWARDED_FOR']

:$_SERVER['REMOTE_ADDR'];

[kcmajor36]

PHP version 5.2.13

[kcmajor36]

and it shows an error every time I put that in there so if you want to fix it your self and post the WHOLE code then that would be great!

[Djkanna]

ereg/i = no no!

Preg_* is a better option.

[kcmajor36]

???

[rulerofzu]

ereg is depreciated as of php 5.3

[Djkanna]

ereg() eregi() are deprecated now (and I think removed as of PHP 6.0)

preg_match()

preg_replace()

Are your better options :)

EDIT: Didn't see RulerOfZu's post O.o

[Danny696]

How hard can it be to replace the whole of

$IP = ($_SERVER['HTTP_X_FORWARDED_FOR'])

? $_SERVER['HTTP_X_FORWARDED_FOR']

:$_SERVER['REMOTE_ADDR'];

with $IP = $_SERVER['REMOTE_ADDR'];

[kcmajor36]

Thats to difficalt could you post the code with them in it?

[kcmajor36]

How hard can it be to replace the whole of

$IP = ($_SERVER['HTTP_X_FORWARDED_FOR'])

? $_SERVER['HTTP_X_FORWARDED_FOR']

:$_SERVER['REMOTE_ADDR'];

with $IP = $_SERVER['REMOTE_ADDR'];

Oh I get it now! ;) Sorry

[kcmajor36]

It works too!

[Danny696]

It works too!

Update the script then ;)

[rulerofzu]

The whole email check function was posted on a linux forum back in 2007

May interest you to find that it was updated to the following. Not tested it though.

 

You're unable to view this code.

Viewing code within this forum requires registration, you can register here for free.

[bladewolf2010]

Can't really call a script you got from google yours :)

[Danny696]

Where did he claim it was his :/