Anyone think this is a good idea?

Corporal Joined: Aug 20, 2003 Posts: 61

I added a TINYTEXT field to nuke_users with the name user_ip. Then in mainfile.php in the online function I added (everything from "else{" on):

Code:

if (!isset($uname)) {
$uname = "$ip";
$guest = 1;
}else {
$result = $db -> sql_query("SELECT user_ip FROM ".$prefix."_users WHERE uname='$uname'");
list($user_ip) = $db -> sql_fetchrow($result);
if ($user_ip != $ip) {
$sql = "UPDATE ".$prefix."_users SET user_ip='$ip' WHERE username='$uname'";
$db -> sql_query($sql);
}
}

This means each time a user logs in it will update their last IP in the usertable. If you have a problem with a user, you always have the last IP they used. Sure someone could spoof or something, but I thought it would be just one more way to track back on someone if necessary.

--Hadron

Posted: Sun Aug 24, 2003 10:20 pm

It is a good idea - you are right about the spoofing and also proxy users at one location will have only one IP address, which could yield multiple users with the same IP, but I love the tracking.

Instead of editing mainfile.php, is there a way to add this as a mod or an include? I couldn't get over how many manual edits were made to mainfile.php over the last few mods I have looked at.

Its not a matter of 'IF' they can get in. Its a matter of what you have in place to track them when they do.

-sting

Corporal Joined: Aug 20, 2003 Posts: 61

You could easily add the same code to the login function of Your_Account or even in the theme.php file. Heck, you can put it most anywhere you know will get accessed (like a User Info Block) every time they visit the site.

Most every Site Visitors block has a similar "if (!isset($uname)) {" code line from what I've seen since they have to test if the user is logged in so they can show the login fields.

--Hadron

Nuke Cops Founder Joined: Nov 14, 2002 Posts: 5939

Hi, I installed a user tracking module into our bundle that captures wonderful user data for each page visit.