From: js6pak <me@6pak.dev>
Date: Fri, 23 Aug 2024 18:57:05 +0000 (+0200)
Subject: Fix RemovePlayer reason on internal disconnects
X-Git-Tag: v1.10.0~2
X-Git-Url: https://git.deb.at/?a=commitdiff_plain;h=acc4da121093dcccb9501bde17b742dc51a60291;p=rhonda%2Fimpostor.git

Fix RemovePlayer reason on internal disconnects
---

diff --git a/src/Impostor.Server/Net/Client.cs b/src/Impostor.Server/Net/Client.cs
index c5c4d16..25d1054 100644
--- a/src/Impostor.Server/Net/Client.cs
+++ b/src/Impostor.Server/Net/Client.cs
@@ -361,7 +361,9 @@ namespace Impostor.Server.Net
             {
                 if (Player != null)
                 {
-                    await Player.Game.HandleRemovePlayer(Id, DisconnectReason.ExitGame);
+                    // The client never sends over the real disconnect reason so we always assume ExitGame
+                    var isRemote = reason == "The remote sent a disconnect request";
+                    await Player.Game.HandleRemovePlayer(Id, isRemote ? DisconnectReason.ExitGame : DisconnectReason.Error);
                 }
             }
             catch (Exception ex)