SquadJS icon indicating copy to clipboard operation
SquadJS copied to clipboard

Update: Admin List Util to Recognize EOSIDs

Open IgnisAlienus opened this issue 1 year ago • 7 comments

So, I updated my admins.cfg to have EOSIDs for my Admins after that whole EOS Update, guess SquadJS hasn't been pulling them this whole time. Now it does.

IgnisAlienus avatar Feb 23 '24 23:02 IgnisAlienus

How does this work when utilising the list? A lot of the Server class seems to search the list of admins by Steam IDs still, e.g.: https://github.com/Team-Silver-Sphere/SquadJS/blob/9f9755ed059de608389229e1b72b3425e8b22e4c/squad-server/index.js#L357

Thomas-Smyth avatar Apr 04 '24 18:04 Thomas-Smyth

@Thomas-Smyth I'm building a full migration to eosIDs. I'm planning to implement a mapping function that will convert steamIDs of online admins into eosIDs (non-matching IDs will be ignored instead of failing). This will allow us to support old steamID-based admin lists while people shift their stuff to eosIDs. My future PR builds upon this PR and my own: https://github.com/Team-Silver-Sphere/SquadJS/pull/352

Ulibos avatar Apr 04 '24 19:04 Ulibos

How does this work when utilising the list? A lot of the Server class seems to search the list of admins by Steam IDs still, e.g.:

https://github.com/Team-Silver-Sphere/SquadJS/blob/9f9755ed059de608389229e1b72b3425e8b22e4c/squad-server/index.js#L357

it accepts both steamID and eosID

fantinodavide avatar Apr 04 '24 19:04 fantinodavide

How does this work when utilising the list? A lot of the Server class seems to search the list of admins by Steam IDs still, e.g.: https://github.com/Team-Silver-Sphere/SquadJS/blob/9f9755ed059de608389229e1b72b3425e8b22e4c/squad-server/index.js#L357

it accepts both steamID and eosID

How? If it's always looking up by Steam ID then it will never match on a EOSID.

Thomas-Smyth avatar Apr 04 '24 20:04 Thomas-Smyth

How does this work when utilising the list? A lot of the Server class seems to search the list of admins by Steam IDs still, e.g.: https://github.com/Team-Silver-Sphere/SquadJS/blob/9f9755ed059de608389229e1b72b3425e8b22e4c/squad-server/index.js#L357

it accepts both steamID and eosID

How? If it's always looking up by Steam ID then it will never match on a EOSID.

People will have to adjust their plugins in order to use eosID, but this PR improves how the admin lists are parsed and won't skip the lines with an eosID only, and doesn't make a specific discrimination between steamID and eosID

fantinodavide avatar Apr 04 '24 20:04 fantinodavide

How? If it's always looking up by Steam ID then it will never match on a EOSID.

If an ID is used directly in rcon commands then it will work regardless. There are 2 plugins that will ignore eosIDs because they loop through players matching admin IDs against steamID:

  1. auto-kick-unassigned.js
  2. discord-admin-request.js

But I'm planning to address this in my next PR.

Ulibos avatar Apr 04 '24 20:04 Ulibos

Pending further review.

werewolfboy13 avatar Apr 05 '24 07:04 werewolfboy13