[jpwanabe]

Page says 1.4.11

[midnight9]

Hmmm, weird. It doesnt work for me at all with 1.4.11. Which version of SourceSleuth are you running? 1.3?

[jpwanabe]

I can verify after updating this plugin, it no longer works.

[FallenNT]

Originally Posted by nomy Change line 111 to: GetClientAuthId(client, AuthId_Steam2, steamid, sizeof(steamid));

Thank compilation took place without errors

[LiMaaa]

Originally Posted by Phaiz I updated to the *fixed TF2 version on your repo but this is still banning players for 120 months Here is my config // This file was auto-generated by SourceMod (v1.7.3-dev+5214) // ConVars for plugin "SourceSleuth.smx" // Sleuth Ban Type: 1 - Original Length, 2 - Custom Length, 3 - Double Length, 4 - Notify Admins Only // - // Default: "3" // Minimum: "1.000000" // Maximum: "4.000000" sm_sleuth_actions "1" // 0 - Inactivated, 1 - Allow all admins with ban flag to pass the check // - // Default: "0" // Minimum: "0.000000" // Maximum: "1.000000" sm_sleuth_adminbypass "0" // How many active bans are allowed before we act // - // Default: "0" sm_sleuth_bansallowed "0" // 0 - ban all type of lengths, 1 - ban only permanent bans // - // Default: "0" // Minimum: "0.000000" // Maximum: "1.000000" sm_sleuth_bantype "0" // Required: sm_sleuth_actions 1: Bantime to ban player if we got a match (0 = permanent (defined in minutes) ) // - // Default: "0" sm_sleuth_duration "1440" // Prexfix for sourcebans tables: Default sb // - // Default: "sb" sm_sleuth_prefix "sb"

I am having the same issue, when a user is banned and tries to access our servers on an alt it bans them for 120months, it isnt always 120months it seems to be multiplying the original ban by something like 21000 times the original length.

I am running: latest Sourcesleuth.
Sourcemod: SourceMod (1.7.3-dev+525 by AlliedModders LLC

[jpwanabe]

change sm_sleuth_bansallowed to 1

[404UserNotFound]

See, here's the issue:

Spoiler

PHP Code:

public SQL_CheckHim(Handle:owner, Handle:hndl, const String:error[], any:datapack)
{
    new client;
    new String:steamid[32], String:IP[32], String:Reason[255], String:text[255];
    
    if(datapack != INVALID_HANDLE)
    {
        client = GetClientOfUserId(ReadPackCell(datapack));
        ReadPackString(datapack, steamid, sizeof(steamid));
        ReadPackString(datapack, IP, sizeof(IP));
        CloseHandle(datapack); 
    }
    
    if (hndl == INVALID_HANDLE)
    {
        LogError("SourceSleuth: Database query error: %s", error);
    }
    
    if (SQL_FetchRow(hndl))
    {
        new TotalBans = SQL_GetRowCount(hndl);
        
        if(TotalBans > GetConVarInt(g_cVar_bansAllowed))
        {
            switch (GetConVarInt(g_cVar_actions))
            {
                case 1:
                {
                    new length = SQL_FetchInt(hndl, 6);
                    new time = length*60;
                    
                    Format(Reason, sizeof(Reason), "[SourceSleuth] %t", "sourcesleuth_banreason");
                    
                    SBBanPlayer(0, client, time, Reason);
                }
                case 2:
                {
                    new time = GetConVarInt(g_cVar_banduration);

                    Format(Reason, sizeof(Reason), "[SourceSleuth] %t", "sourcesleuth_banreason");
                    
                    SBBanPlayer(0, client, time, Reason);
                }
                case 3:
                {
                    new length = SQL_FetchInt(hndl, 6);
                    new time = length/60*2;

                    Format(Reason, sizeof(Reason), "[SourceSleuth] %t", "sourcesleuth_banreason");
                    
                    SBBanPlayer(0, client, time, Reason);
                }
                case 4:
                {
                    Format(text, sizeof(text), "[SourceSleuth] %t", "sourcesleuth_admintext",client, steamid, IP);
                    PrintToAdmins("%s", text);
                }
            }
        }
    }
} 


There's no option for keeping the existing ban time which would alleviate this issue. I've made a quick fix to this plugin to add a 5th option but I'm not sure if it'll work (it needs testing):

Spoiler

PHP Code:

public SQL_CheckHim(Handle:owner, Handle:hndl, const String:error[], any:datapack)
{
    new client;
    new String:steamid[32], String:IP[32], String:Reason[255], String:text[255];
    
    if(datapack != INVALID_HANDLE)
    {
        client = GetClientOfUserId(ReadPackCell(datapack));
        ReadPackString(datapack, steamid, sizeof(steamid));
        ReadPackString(datapack, IP, sizeof(IP));
        CloseHandle(datapack); 
    }
    
    if (hndl == INVALID_HANDLE)
    {
        LogError("SourceSleuth: Database query error: %s", error);
    }
    
    if (SQL_FetchRow(hndl))
    {
        new TotalBans = SQL_GetRowCount(hndl);
        
        if(TotalBans > GetConVarInt(g_cVar_bansAllowed))
        {
            switch (GetConVarInt(g_cVar_actions))
            {
                case 1:
                {
                    new length = SQL_FetchInt(hndl, 6);
                    new time = length*60;
                    
                    Format(Reason, sizeof(Reason), "[SourceSleuth] %t", "sourcesleuth_banreason");
                    
                    SBBanPlayer(0, client, time, Reason);
                }
                case 2:
                {
                    new time = GetConVarInt(g_cVar_banduration);

                    Format(Reason, sizeof(Reason), "[SourceSleuth] %t", "sourcesleuth_banreason");
                    
                    SBBanPlayer(0, client, time, Reason);
                }
                case 3:
                {
                    new length = SQL_FetchInt(hndl, 6);
                    new time = length/60*2;

                    Format(Reason, sizeof(Reason), "[SourceSleuth] %t", "sourcesleuth_banreason");
                    
                    SBBanPlayer(0, client, time, Reason);
                }
                case 4:
                {
                    Format(text, sizeof(text), "[SourceSleuth] %t", "sourcesleuth_admintext",client, steamid, IP);
                    PrintToAdmins("%s", text);
                }
                case 5:
                {
                    new length = SQL_FetchInt(hndl, 6);
                    new time = length;

                    Format(Reason, sizeof(Reason), "[SourceSleuth] %t", "sourcesleuth_banreason");
                    
                    SBBanPlayer(0, client, time, Reason);
                }
            }
        }
    }
} 


So if you decide to give this version a try, set "sm_sleuth_actions" to "5" and let me know if it rebans a reconnecting offender under the same original ban length, or if it starts multiplying things again.

[jpwanabe]

Still can't figure out why this isn't working for me. Can the people who have it working post what OS they are on and maybe their cfg. I wonder if that may be a problem.

[Tranced]

Originally Posted by jpwanabe Still can't figure out why this isn't working for me. Can the people who have it working post what OS they are on and maybe their cfg. I wonder if that may be a problem.

Have you by any chance got it figured it out? I'm running it on my linux servers without an issue.

[jpwanabe]

Originally Posted by Tranced Have you by any chance got it figured it out? I'm running it on my linux servers without an issue.

Nope. Still doesn't work for me.