ect0s

That works. I was trying to sort the player controller situation out in the log parser, and needed that to persist across maps. Player Controllers are *unique* to each player...

We fire 'PLAYER_DISCONNECTED' when EAC unregisters a client, which happens on map changes. But this event doesn't actually mean the player disconnected; as they may reregister for the new map....

https://github.com/ect0s/SquadJS/blob/Player-Disconnected-Fix/squad-server/log-parser/player-disconnected.js Found a better regex for player disconnected

These change were successfully tested on TT this weekend. As of right now, there doesn't appear to need to be a delay on NEW_GAME. I did notice one issue, with...

This has been running stable at TT for some time. I'm not sure if anything needs to be done, but it should be viable to merge.

It appears I missed the early return for draws > Got this error on mine when I added it.  This should be fixed

``` [2022.09.29-10.47.59:424][474]LogSquadGameEvents: Display: Team 2, RUS 49th Combined Arms Army ( Russian Ground Forces ) has won the match with 294 Tickets on layer Narva RAAS v5 (level Narva)! [2022.09.29-10.47.59:424][474]LogSquadGameEvents:...

> Is that really the case? Yes, the above log lines are directly from testing. In the past, before OWI added the 'victory flags' animations to the scoreboard, you would...

https://github.com/Team-Silver-Sphere/SquadJS/blob/dc2bc1d663fb5b4aecab39d9072ef51dad089fae/core/rcon.js#L172 Is this ternary backwards? Close() with out data will result in the false case; leading to "with an Error"

> Yeah, that's backwards. > > I checked the node documentation to rule out, that the parameter was poorly named. https://nodejs.org/api/net.html#event-close_1 > > > Event: 'close' > > Added in:...