[Bacardi]

Originally Posted by Forlix What does that mean?

Those bans what have time, not permit...
Unban remove ban from server list but not from "Forlix TemBanfix list"
Plugin keep still ban activated
I tried mapchange if that help

[Bacardi]

Ok, I tried again.. I banned myself by steam_id 30minutes
- Looking list bans rcon listid not show any temp bans...
- Using rcon to unban not work..
rcon sm_unban steam_id
rcon removeid slot
rcon removeid steam_id

- Only way remove temp ban is type to straight srcds.exe console input
removeid steam_id

[Forlix]

 
sm_unban STEAM_0:0:27017xxx
L 06/27/2010 - 18:15:43: [basebans.smx] "Console<0><Console><Console>" removed ban (filter "STEAM_0:0:27017xxx")
[SM] Removed bans matching filter: STEAM_0:0:27017xxx
removeid (TempBanFix): filter removed for STEAM_0:0:27017xxx
Writing cfg/banned_user.cfg.

Works for me...

[Forlix]

Check this out:

] rcon sm_unban STEAM_0:0:27017xxx
L 06/27/2010 - 18:21:23: [basebans.smx] "Console<0><Console><Console>" removed ban (filter "STEAM_0 : 0 : 27017xxx")
[SM] Removed bans matching filter: STEAM_0 : 0 : 27017xxx
removeip:  invalid slot 0
Writing cfg/banned_ip.cfg.
] rcon sm_unban "STEAM_0:0:27017xxx"
L 06/27/2010 - 18:21:55: [basebans.smx] "Console<0><Console><Console>" removed ban (filter "STEAM_0:0:27017xxx")
[SM] Removed bans matching filter: STEAM_0:0:27017xxx
removeid (TempBanFix):  filter removed for STEAM_0:0:27017xxx
Writing cfg/banned_user.cfg.

The engine seems to recognize ":" as an argument separator, it only works when you wrap the STEAMID in quotes.
Dont know if this also applies when using sm_unban without rcon.

[Bacardi]

Indeed ! I try go test right a way. Thanks for info

[zeroibis]

So does this play nice with sourcebans?

[GrO]

Originally Posted by Forlix Changes in v1.13: (2011/04) •Added RCON support - Use listidt to list all temporary bans

Great plugin and very useful, thanks for Your good job done here.

There's just one little problem with KAC plugin every time a TempBanned player is trying to join:

L 09/07/2011 - 16:02:24: SourceMod error session started
L 09/07/2011 - 16:02:24: Info (map "de_prodigy") (file "errors_20110907.log")
L 09/07/2011 - 16:02:24: [SM] Native "IsFakeClient" reported: Client 1 is not connected
L 09/07/2011 - 16:02:24: [SM] Displaying call stack trace for plugin "kigen-ac-pub.smx":
L 09/07/2011 - 16:02:24: [SM]   [0]  Line 250, kigen-ac-pub.sp::OnClientAuthorized()
L 09/07/2011 - 16:14:24: Error log file session closed.
L 09/07/2011 - 19:07:32: SourceMod error session started
L 09/07/2011 - 19:07:32: Info (map "de_cbble") (file "errors_20110907.log")
L 09/07/2011 - 19:07:32: [SM] Native "IsFakeClient" reported: Client 14 is not connected
L 09/07/2011 - 19:07:32: [SM] Displaying call stack trace for plugin "kigen-ac-pub.smx":
L 09/07/2011 - 19:07:32: [SM]   [0]  Line 250, kigen-ac-pub.sp::OnClientAuthorized()
L 09/07/2011 - 19:12:03: Error log file session closed.

Is there any way to make Your plugin to refuse players alittle bit later or something I can change/add in/to KAC's code to avoid this error?

[Forlix]

You should add a check that determines via IsClientConnected if the client is connected before calling IsFakeClient at those places - that will make the error go away. If you need help with that, post the lines of code around the IsFakeClient calls, and I can add it in for you.

[GrO]

Originally Posted by Forlix You should add a check that determines via IsClientConnected if the client is connected before calling IsFakeClient at those places - that will make the error go away. If you need help with that, post the lines of code around the IsFakeClient calls, and I can add it in for you.

Thanks for Your help ;]

I hope this is enaugh, cause there are more lines with IsFakeClient also with one of includes, but this is the part mentioned in the error log:

public OnClientAuthorized(client, const String:auth[])
{
	if ( IsFakeClient(client) ) // Bots are annoying...
		return;

	decl Handle:f_hTemp, String:f_sReason[256];

	if ( GetTrieString(g_hDenyArray, auth, f_sReason, sizeof(f_sReason)) )
	{
		KickClient(client, "%s", f_sReason);
		OnClientDisconnect(client);
		return;
	}

[Forlix]

Replace this

    if ( IsFakeClient(client) ) // Bots are annoying...
        return;

with:

    if ( !IsClientConnected(client)
    ||   IsFakeClient(client) ) // Bots are annoying...
        return;

That should do it.