mirror of
https://github.com/Sanae6/SmoOnlineServer.git
synced 2024-11-10 13:45:06 +00:00
only start listening for clients once everything is initialized
Otherwise clients might connect to the server before everything is ready for them. E.g. when restarting the server, the clients will immediately try to reconnect. Clients might connect before the `PacketHandler` is initialized, which results in some packets not being processed by the server. Same goes for the commands: Discord or the JsonApi might send in commands before all commands were added to the `CommandHandler`. Without the `ClientJoined` action, clients might even be allowed connect if they are on the banlist.
This commit is contained in:
parent
87cec91941
commit
92e540aaa6
1 changed files with 2 additions and 2 deletions
|
@ -12,7 +12,6 @@ Server.Server server = new Server.Server();
|
|||
HashSet<int> shineBag = new HashSet<int>();
|
||||
CancellationTokenSource cts = new CancellationTokenSource();
|
||||
bool restartRequested = false;
|
||||
Task listenTask = server.Listen(cts.Token);
|
||||
Logger consoleLogger = new Logger("Console");
|
||||
DiscordBot bot = new DiscordBot();
|
||||
await bot.Run();
|
||||
|
@ -655,7 +654,8 @@ Task.Run(() => {
|
|||
}).ContinueWith(x => { if (x.Exception != null) { consoleLogger.Error(x.Exception.ToString()); } });
|
||||
#pragma warning restore CS4014
|
||||
|
||||
await listenTask;
|
||||
await server.Listen(cts.Token);
|
||||
|
||||
if (restartRequested) //need to do this here because this needs to happen after the listener closes, and there isn't an
|
||||
//easy way to sync in the restartserver command without it exiting Main()
|
||||
{
|
||||
|
|
Loading…
Reference in a new issue