search

gekware / minecraft-server-hibernation

Autostart and stop minecraft-server when players join/leave
GNU General Public License v3.0
389 stars 38 forks source link

`countPlayerSafe` fails at first execution after ms start #174

Closed gekigek99 closed 1 year ago

gekigek99 commented 1 year ago

Current Behavior (🐛 Bug)

in title

Expected Behavior

countPlayerSafe should work at first execution after ms start

Steps to Reproduce

msh start mine msg "i think he lost connection" (1 time) mine msg "i think he lost connection" (2 time)

MSH Logs

first call of countPlayerSafe after ms became online:

2022/12/02 00:32:03 [info  ≡≡≡ ] MINECRAFT SERVER IS ONLINE!
2022/12/02 00:32:03 [info  ≡   ] scheduling ms soft freeze in 300 seconds
2022/12/02 00:32:03 [serv  ≡≡  ] [00:32:03] [Server thread/INFO]: Starting remote control listener
2022/12/02 00:32:03 [serv  ≡≡  ] [00:32:03] [Server thread/INFO]: Thread RCON Listener started
2022/12/02 00:32:03 [serv  ≡≡  ] [00:32:03] [Server thread/INFO]: RCON running on 0.0.0.0:25575
mine msg "i think he lost connection"
2022/12/02 00:32:11 [info  ≡   ] user input: [mine msg "i think he lost connection"]
2022/12/02 00:32:11 [info  ≡≡  ] ms command: msg "i think he lost connection"   (origin: GetInput)
2022/12/02 00:32:11 [serv  ≡≡  ] [00:32:11] [Server thread/INFO]: Invalid name or UUID
2022/12/02 00:32:11 [serv  ≡≡  ] [00:32:11] [Server thread/INFO]: msg "i think he lost connection"<--[HERE]
2022/12/02 00:32:11 [info  ≡   ] wrong playercount (-1), resetting it from a more reliable source
2022/12/02 00:32:11 [info  ≡   ] retrieving player count...
2022/12/02 00:32:11 [error ≡   ] countPlayerSafe: getPlayersByServInfo: getServInfo: received data unexpected format ([]) [02f201]
2022/12/02 00:32:11 [info  ≡≡  ] ms command: list       (origin: getPlayersByListCom)
2022/12/02 00:32:11 [error ≡   ] countPlayerSafe: getPlayersByListCom: StrBetween: first substring not found (" of a max") [05f000]
2022/12/02 00:32:11 [info  ≡≡≡ ] -1 online players - method for player count: internal
2022/12/02 00:32:11 [info  ≡≡≡ ] A PLAYER LEFT THE SERVER! - -1 players online
2022/12/02 00:32:11 [info  ≡   ] scheduling ms soft freeze in 300 seconds
2022/12/02 00:32:11 [serv  ≡≡  ] [00:32:11] [Server thread/INFO]: There are 0 of a max of 20 players online:

second call of countPlayerSafe after ms became online

mine msg "i think he lost connection"
2022/12/02 00:32:16 [info  ≡   ] user input: [mine msg "i think he lost connection"]
2022/12/02 00:32:16 [info  ≡≡  ] ms command: msg "i think he lost connection"   (origin: GetInput)
2022/12/02 00:32:16 [serv  ≡≡  ] [00:32:16] [Server thread/INFO]: Invalid name or UUID
2022/12/02 00:32:16 [serv  ≡≡  ] [00:32:16] [Server thread/INFO]: msg "i think he lost connection"<--[HERE]
2022/12/02 00:32:16 [info  ≡   ] wrong playercount (-2), resetting it from a more reliable source
2022/12/02 00:32:16 [info  ≡   ] retrieving player count...
2022/12/02 00:32:16 [info  ≡≡≡ ] 0 online players - method for player count: server info
2022/12/02 00:32:16 [info  ≡≡≡ ] A PLAYER LEFT THE SERVER! - 0 players online
2022/12/02 00:32:16 [info  ≡   ] scheduling ms soft freeze in 300 seconds

MSH-Version

2.4.9

Operating system

win 10

Minecraft Version

1.19.2

Minecraft Server Software

vanilla

Is there an existing issue for this?

gekigek99 commented 1 year ago

fixed in a490f1ec3b9691473db1ba94e81f51e082f6b47f