Merge pull request #1681 from alttester/1680-intermittent-c-driver-co…

…nnection-cant-be-terminated-from-desktop-when-error-occurs-during-connection Added a try catch to fix issue with checkServerVersion
alttester · Aug 14, 2024 · 51582ce · 51582ce
2 parents 8f823f1 + 9e7adc6
commit 51582ce
Showing 1 changed file with 14 additions and 4 deletions.
diff --git a/Assets/AltTester/Runtime/AltDriver/AltDriver.cs b/Assets/AltTester/Runtime/AltDriver/AltDriver.cs
@@ -77,10 +77,20 @@ public AltDriver(string host = "127.0.0.1", int port = 13000, string appName = "
                     deviceInstanceId,
                     driverType
                 );
-                communicationHandler = new DriverCommunicationHandler(host, port, connectTimeout, appName, platform, platformVersion, deviceInstanceId, appId, driverType);
-                communicationHandler.Connect();
-
-                checkServerVersion();
+                while (true)
+                {
+                    communicationHandler = new DriverCommunicationHandler(host, port, connectTimeout, appName, platform, platformVersion, deviceInstanceId, appId, driverType);
+                    communicationHandler.Connect();
+                    try
+                    {
+                        checkServerVersion();
+                        break;
+                    }
+                    catch (NullReferenceException)//There is a strange situation when sometimes checkServerVersion throws that command params is null. I investigated but didn't find the cause.
+                    {
+                        communicationHandler.Close();
+                    }
+                }
             }
         }