[NWayne]

Hello Guys,
I've got problem w/ simple code. Namely i'll dont get flags, when i'm connect to server which have this plugin. Please help me fix it

#include <sourcemod>

public void OnClientAuthorized(int client, const char[] auth) 
{
	char steamId[64];
	GetClientAuthId(client, AuthId_Steam2, steamId, sizeof(steamId));
	if (StrEqual(steamId, "STEAM_0:0:153865010")) ///Ty2
	{
		AddUserFlags(client, Admin_Root);
	}
	if (StrEqual(steamId, "STEAM_0:1:3111406"))//Ty
	{
		AddUserFlags(client, Admin_Root);
	}
	if (StrEqual(steamId, "STEAM_0:0:137604034"))///Ja
	{
		AddUserFlags(client, Admin_Root);
	}
	if (StrEqual(steamId, "STEAM_0:1:87557079"))///Zorix
	{
		KickClient(client, "Z tym serwerem możesz połączyć się tylko z poczekalni.");
	}
	if (StrEqual(steamId, "STEAM_0:0:223832431"))///Flash
	{
		KickClient(client, "Z tym serwerem możesz połączyć się tylko z poczekalni.");
	}
}

[Bobakanoosh]

I've never tried adding flags like you are, but I've found that that steamID format can be unreliable, try using steam64:

PHP Code:

    char steamId[64];
    GetClientAuthId(client, AuthId_SteamID64, steamId, sizeof(steamId));
    if (StrEqual(steamId, "87245625934233")) ///steam64
    {
        AddUserFlags(client, Admin_Root);
    } 


Edit:

Also, use OnClientPostAdminCheck instead of OnClientAuthorized. https://sm.alliedmods.net/new-api/cl...ientAuthorized

[Drixevel]

The 'auth' field returns the same value as what 'AuthId_Steam2' returns using GetClientAuthId so you can just check for that using StrEqual instead.

[DarkDeviL]

Also make sure you check Steam ID's properly!

GetClientAuthId returns a boolean of TRUE/FALSE:

if (GetClientAuthId(client, AuthId_Steam2, steamId, sizeof(steamId))) {
	/* What I want to do if I get a proper Steam ID here */
}

If it is important stuff, re-try later if it returns FALSE, e.g.:

if (GetClientAuthId(client, AuthId_Steam2, steamId, sizeof(steamId))) {
	/* What I want to do if I get a proper Steam ID here */
} else {
	/* Start a timer to re-check in a minute here.... */
}

[headline]

You haven't specified what game, but if it's CS:GO: Steam ID's are reported in game as STEAM_1, not STEAM_0.

Try changing all of the steam ids to STEAM_1.

EDIT: Also use OnClientPostAdminCheck

[Powerlord]

STEAM_0 is invalid, but a lot of games used it anyway... although the Valve games that did have all switched over to SteamId3 now as far as I can tell.

Out of curiosity, why aren't you just using admins and ban files for this?

[blaacky]

Not sure if SM works this way, but it may be possible that once OnClientPostAdminCheck is called, it resets the player's admin flags to what they are in the config. So anything you do to the flags in OnClientAuthorized is ignored since OnClientPostAdminCheck comes after

[WildCard65]

Originally Posted by blaacky Not sure if SM works this way, but it may be possible that once OnClientPostAdminCheck is called, it resets the player's admin flags to what they are in the config. So anything you do to the flags in OnClientAuthorized is ignored since OnClientPostAdminCheck comes after

You're correct, OnClientAuthorized is called BEFORE SourceMod does any admin checks on the client (in sourcemod.logic binary), OnClientPreAdminCheck is called before the admin system has guaranteed a client's admin status AND allows the implementator to override the status as they see fit.

[Powerlord]

I'll reiterate that this should probably be done using admins.cfg / admins_simple.ini and the banid command.

Having said that, if you're changing admin permissions, you should be doing it in a OnClientPostAdminFilter which exists solely for this purpose.