fixed issue handling challenge/response for A2S_PLAYERS and A2S_RULES

PredatH0r · Sep 4, 2024 · 0a968eb · 0a968eb
1 parent 25b849f
commit 0a968eb
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 14 deletions.
diff --git a/QueryMaster/QueryMaster/Server.cs b/QueryMaster/QueryMaster/Server.cs
@@ -284,14 +284,12 @@ protected virtual ReadOnlyCollection<Player> GetPlayersCore(Stopwatch sw = null)
       }
       else
       {
-        if (PlayerChallengeId == null)
-        {
-          recvData = GetPlayerChallengeId(sw);
-          if (IsPlayerChallengeId)
-            PlayerChallengeId = recvData;
-        }
+        recvData = GetPlayerChallengeId(sw);
         if (IsPlayerChallengeId)
+        {
+          PlayerChallengeId = recvData;
           recvData = socket.GetResponse(Util.MergeByteArrays(QueryMsg.PlayerQuery, PlayerChallengeId), Type);
+        }
       }
       try
       {
@@ -340,14 +338,12 @@ protected virtual ReadOnlyCollection<Rule> GetRulesCore()
       }
       else
       {
-        if (RuleChallengeId == null)
-        {
-          recvData = GetRuleChallengeId();
-          if (IsRuleChallengeId)
-            RuleChallengeId = recvData;
-        }
+        recvData = GetRuleChallengeId();
         if (IsRuleChallengeId)
+        {
+          RuleChallengeId = recvData;
           recvData = socket.GetResponse(Util.MergeByteArrays(QueryMsg.RuleQuery, RuleChallengeId), Type);
+        }
       }
       try
       {

diff --git a/ServerBrowser/Games/Toxikk.cs b/ServerBrowser/Games/Toxikk.cs
@@ -380,7 +380,7 @@ private void ConnectInBackground(ServerRow server, string password, bool spectat
       // send the command string
       var msg = "open ";
       bool steamSockets = server.Rules == null || !string.IsNullOrEmpty(server.GetRule("SteamServerId"));
-      if ((useSteamIdToConnect && steamSockets || server.Rules == null || server.Players == null) && server.ServerInfo.Extra.SteamID != 0)
+      if ((useSteamIdToConnect && steamSockets /* || server.Rules == null || server.Players == null */) && server.ServerInfo.Extra.SteamID != 0)
         msg += "steam." + server.ServerInfo.Extra.SteamID;
       else
         msg += server.EndPoint.Address + ":" + server.ServerInfo.Extra.Port;

diff --git a/ServerBrowser/ServerBrowserForm.cs b/ServerBrowser/ServerBrowserForm.cs
@@ -33,7 +33,7 @@ namespace ServerBrowser
 {
   public partial class ServerBrowserForm : XtraForm
   {
-    private const string Version = "2.67";
+    private const string Version = "2.68";
     private const string DevExpressVersion = "v23.2";
     private const string OldSteamWebApiText = "<Steam Web API>";
     private const string CustomDetailColumnPrefix = "ServerInfo.";

diff --git a/changelog.md b/changelog.md
@@ -1,3 +1,7 @@
+2.68
+---
+- fixed issues with missing Rules and Players information due to bad challenge/response handling
+
 2.67
 ---
 - fixed typo in "throtteling"