Fixes maintainer access to the profiler and clears profiler access whenever admins are loaded.

[Watermelon914]

About the pull request

As explained by the title.

Explain why it's good for the game

Prevents a vulnerability where someone who previously had access to the profiler retains access despite no longer requiring it.

No player facing change so changelog not needed

[Drulikar]

I do not understand why on event server this works (after a reconnect - not first connect); but not on live (even with reconnects). I can confirm for both servers the cfg/admin.txt is assigned to be some_ckey role=admin for only those who have connected this round, and the perms for the file are the same for both servers.

But the aspect of requiring a reconnect is wonky as is. So try setting up the admin.txt during init (e.g. after load_admin_ranks) rather than on associate?

[Watermelon914]

I do not understand why on event server this works (after a reconnect - not first connect); but not on live (even with reconnects). I can confirm for both servers the cfg/admin.txt is assigned to be some_ckey role=admin for only those who have connected this round, and the perms for the file are the same for both servers.

But the aspect of requiring a reconnect is wonky as is. So try setting up the admin.txt during init (e.g. after load_admin_ranks) rather than on associate?

I was doing testing with harryob, and the admin.txt was getting populated with ckeys. The problem isn't related to code, it's related to the cm-live server and it's unclear what might be causing it. In any case, this PR is still useful to merge since it clears up a potential vulnerability where people stay in the admin.txt despite probably not being an admin anymore.