SP-2359: Do not crash in constructor if config file cannot be upgrade…

.github/workflows/PR.yml

@@ -1,6 +1,7 @@
 on:
   pull_request:
     branches: [ master ]
+
 jobs:
   build:
     strategy:
@@ -14,6 +15,14 @@ jobs:
       with:
         fetch-depth: 0
 
+    - name: Install libssl1.1 on Ubuntu
+      if: matrix.os == 'ubuntu-latest'
+      run: |
+        sudo apt-get update
+        sudo apt-get install -y wget
+        wget http://security.ubuntu.com/ubuntu/pool/main/o/openssl1.1/libssl1.1_1.1.1f-1ubuntu2.19_amd64.deb
+        sudo dpkg -i libssl1.1_1.1.1f-1ubuntu2.19_amd64.deb
+
     - name: Install .NET Core
       uses: actions/setup-dotnet@77a48bbb85cd3809790223e895ba4b6cbc110fe8
       with:
@@ -24,4 +33,3 @@ jobs:
 
     - name: Test
       run: dotnet test
-

src/DesktopAnalytics/Analytics.cs

@@ -10,6 +10,7 @@
 using System.Net;
 using System.Reflection;
 using System.Runtime.InteropServices;
+using System.Threading;
 using System.Xml.Linq;
 using System.Xml.XPath;
 using JetBrains.Annotations;
@@ -172,9 +173,23 @@ public Analytics(string apiSecret, UserInfo userInfo, Dictionary<string,
 			if (AnalyticsSettings.Default.NeedUpgrade)
 			{
 				//see http://stackoverflow.com/questions/3498561/net-applicationsettingsbase-should-i-call-upgrade-every-time-i-load
-				AnalyticsSettings.Default.Upgrade();
-				AnalyticsSettings.Default.NeedUpgrade = false;
-				AnalyticsSettings.Default.Save();
+				try
+				{
+					AnalyticsSettings.Default.Upgrade();
+					AnalyticsSettings.Default.NeedUpgrade = false;
+					TrySaveSettings();
+				}
+				catch (ConfigurationErrorsException e)
+				{
+					try
+					{
+						Console.WriteLine(e);
+					}
+					catch
+					{
+						Debug.WriteLine(e);
+					}
+				}
 			}
 
 			if (IsNullOrEmpty(AnalyticsSettings.Default.IdForAnalytics))
@@ -195,9 +210,8 @@ public Analytics(string apiSecret, UserInfo userInfo, Dictionary<string,
 
 			if (IsNullOrEmpty(AnalyticsSettings.Default.IdForAnalytics))
 			{
-
 				AnalyticsSettings.Default.IdForAnalytics = Guid.NewGuid().ToString();
-				AnalyticsSettings.Default.Save();
+				TrySaveSettings();
 			}
 
 			s_locationInfo = new JsonObject();
@@ -236,7 +250,31 @@ public Analytics(string apiSecret, UserInfo userInfo, Dictionary<string,
 			// See http://issues.bloomlibrary.org/youtrack/issue/BL-4011.
 
 			AnalyticsSettings.Default.LastVersionLaunched = versionNumberWithBuild;
-			AnalyticsSettings.Default.Save();
+			TrySaveSettings();
+		}
+
+		private static void TrySaveSettings()
+		{
+			int retryCount = 0;
+			do
+			{
+				try
+				{
+					AnalyticsSettings.Default.Save();
+					return;
+				}
+				catch (Exception e)
+				{
+					try
+					{
+						Console.WriteLine(e);
+					}
+					catch
+					{
+					}
+					Thread.Sleep(300);
+				}
+			} while (++retryCount < 3);
 		}
 
 		private void AttemptToGetUserIdSettingsFromDifferentChannel()
@@ -309,7 +347,7 @@ private void AttemptToGetUserIdSettingsFromDifferentChannel()
 						AnalyticsSettings.Default.LastName = ExtractSetting(AnalyticsSettings.Default.LastName, doc, "LastName");
 						AnalyticsSettings.Default.LastVersionLaunched = ExtractSetting(AnalyticsSettings.Default.LastVersionLaunched, doc, "LastVersionLaunched");
 						AnalyticsSettings.Default.Email = ExtractSetting(AnalyticsSettings.Default.Email, doc, "Email");
-						AnalyticsSettings.Default.Save();
+						TrySaveSettings();
 						return;
 					}
 					catch (Exception)