[Rseding91] [0.15.21] Server crash "Invalid value of AllowedCommands"

[GeneralDotV2]

When using the command "/config set allow-commands" and you set the value to anything that isn't true, false or admins-only, the server will go to a failed state.
I noticed this when i accidentally had a typo in the value and typed "falsea" instead of "false".
When you use the command "/config set require-user-verification falsea", for example, you get a "Value is not valid" message instead of a crash, which i believe "/config set allow-commands" should give the same message instead of closing the server.

To me, this is not a big issue, just something I wanted to point out and made known to the developers.

Code: Select all

62596.850 Error MainLoop.cpp:943: Exception at tick 2141769: Invalid value "falsea" of AllowedCommands
62596.850 Error ServerMultiplayerManager.cpp:94: MultiplayerManager failed: "Invalid value "falsea" of AllowedCommands"
62596.850 Info ServerMultiplayerManager.cpp:670: mapTick(2141769) changing state from(InGame) to(Failed)
62597.980 Info ServerMultiplayerManager.cpp:140: Quitting multiplayer connection.
62597.980 Info ServerMultiplayerManager.cpp:670: mapTick(-1) changing state from(Failed) to(Closed)
62597.980 Info HttpSharedState.cpp:44: Downloading https://multiplayer.factorio.com/remove-game/2392962
62598.088 Info HttpSharedState.cpp:111: Status code: 200
62598.088 Info UDPSocket.cpp:234: Socket closed
62598.120 Info UDPSocket.cpp:234: Socket closed
62598.120 Goodbye

[Rseding91]

Thanks for the report. It's now fixed for then ext version of 0.15.