OcixCrom - You are using TrieGetString() without ever calling TrieSetString(), so the condition in client_authorized() will never be true. It looks like you formatted the string in client_disconnect() in preparation to use it, but forgot.
This comes down to preference, but you can avoid the conditional #if pre-processor directives at the top by just redefining using pre-processor:
#define MAX_PLAYERS 32
#define MAX_NAME_LENGTH 32
#define MAX_AUTHID_LENGTH 64