IP Adress Log

[Tezuni]

Code: [Select]

function gameConnection::onClientEnterGame(%client)
        {
                parent::onClientEnterGame(%client);
        %file = new FileObject();
        %file.openForAppend("config/server/TezLogs/IPAdresses.log");
        %file.writeLine("Player:" SPC %client.name SPC "\nUser ID:" SPC %client.BL_ID @ "\n" @ %client.getAddress() @ "\n-----------------------------------------");
        %file.close();
        %file.delete();
        }
It doesn't seem to work, and also I'm not sure how to make it not log the same person twice.  Help is appreciated, thanks!

[Ad Bot]

[Chrono]

Shouldn't it do %client.getRawIP() ?
Also should be autoAdminCheck, because onClientEnterGame is when they first spawn.

[Tezuni]

Shouldn't it do %client.getRawIP() ?
Also should be autoAdminCheck, because onClientEnterGame is when they first spawn.

So....this?

Code: [Select]

function gameConnection::AutoAdminCheck(%client)
        {
                parent::AutoAdminCheck(%client);
        %file = new FileObject();
        %file.openForAppend("config/server/TezLogs/IPAdresses.log");
        %file.writeLine("Player:" SPC %client.name SPC "\nUser ID:" SPC %client.BL_ID @ "\n" @ %client.getRawIP()  @ "\n-----------------------------------------");
        %file.close();
        %file.delete();
        }

[Chrono]

Yes but just one more thing, just so stuff don't break:

Code: [Select]

function gameConnection::AutoAdminCheck(%client)
        {
                %ret = parent::AutoAdminCheck(%client);
        %file = new FileObject();
        %file.openForAppend("config/server/TezLogs/IPAdresses.log");
        %file.writeLine("Player:" SPC %client.name SPC "\nUser ID:" SPC %client.BL_ID @ "\n" @ %client.getRawIP()  @ "\n-----------------------------------------");
        %file.close();
        %file.delete();
        return %ret;
        }That also returns what AutoAdminCheck would originally return.

[Tezuni]

thank you it, works.

Is there a way to make it not log the same person twice?

[Chrono]

Yeah but I don't feel like writing anything right now.

[Tezuni]

Yeah but I don't feel like writing anything right now.

>:C

[Truce]

>:C

First open the file for reading, and if you read a line that begins with "User ID:" and is followed by their ID, close the file and skip the writing step.

[Spockticus]

Interesting idea.

[Ephialtes]

First open the file for reading, and if you read a line that begins with "User ID:" and is followed by their ID, close the file and skip the writing step.

You'll find a file in that format can grow in size pretty fast, and you'll then find that reading through every line to check for existing IDs really takes some time.

[Truce]

You'll find a file in that format can grow in size pretty fast, and you'll then find that reading through every line to check for existing IDs really takes some time.

True, since he's using a good number of lines per ID.

@Tezuni: An alternative option would be to store logged IDs in a global variable (e.g. $LoggedIP[%client.bl_id]), and export / exec on server statup. You can just use a quick if test to see if they've been logged or not then.

[Tezuni]

@Tezuni: An alternative option would be to store logged IDs in a global variable (e.g. $LoggedIP[%client.bl_id]), and export / exec on server statup. You can just use a quick if test to see if they've been logged or not then.

I've never done that type of thing before, I'm not sure where to begin.  Could you edit the code a bit to show me? Thanks.

[Kalphiter]

You'll find a file in that format can grow in size pretty fast, and you'll then find that reading through every line to check for existing IDs really takes some time.

Hahaha, CityRP.

[Resonance_Cascade]

Code: [Select]

function GameConnection::AutoAdminCheck(%client)
{
   $LoggedIP[%client.BL_ID] = %client.getRawIP();
   return Parent::AutoAdminCheck(%client);
}
You can export the variables using:

Code: [Select]

export("$LoggedIP*", "config/server/TezLogs/IPAdresses.log", false);
and execute them using:

Code: [Select]

exec("config/server/TezLogs/IPAddresses.log");

[Kalphiter]

Code: [Select]

exec("config/server/TezLogs/IPAddresses.log");

Wtf is this?