improve logging on startup services failures (#1962)

hahn-kev · web-flow · commit 8d5aa0495020 · 2025-09-03T14:11:01.000+07:00
* catch and log errors when starting hosted services in the MAUI hosted service adapter

* don't try to install updates when running a portable app

* display a notification in app when the update was successfully installed
diff --git a/backend/FwLite/FwLiteMaui/FwLiteMauiKernel.cs b/backend/FwLite/FwLiteMaui/FwLiteMauiKernel.cs
@@ -165,7 +165,22 @@ public void Initialize(IServiceProvider services)
             logger.LogInformation("Initializing hosted services");
             foreach (var hostedService in hostedServices)
             {
-                _ = Task.Run(() => hostedService.StartAsync(_cts.Token), _cts.Token);
+                _ = Task.Run(async () =>
+                {
+                    var hostedServiceName = hostedService.GetType().Name;
+                    try
+                    {
+                        await hostedService.StartAsync(_cts.Token);
+                    }
+                    catch (OperationCanceledException)
+                    {
+                        logger.LogInformation("Hosted service stopped before start completed for service {HostedService}", hostedServiceName);
+                    }
+                    catch (Exception e)
+                    {
+                        logger.LogError(e, "Error starting hosted service {HostedService}", hostedServiceName);;
+                    }
+                }, _cts.Token);
             }
         }
 
diff --git a/backend/FwLite/FwLiteMaui/Platforms/Windows/AppUpdateService.cs b/backend/FwLite/FwLiteMaui/Platforms/Windows/AppUpdateService.cs
@@ -152,7 +152,7 @@ public DateTime LastUpdateCheck
         set => preferences.Set(LastUpdateCheckKey, value);
     }
 
-    public bool SupportsAutoUpdate => true;
+    public bool SupportsAutoUpdate => !FwLiteMauiKernel.IsPortableApp;
 
     public bool IsOnMeteredConnection()
     {
diff --git a/frontend/viewer/src/lib/notifications/NotificationOutlet.svelte b/frontend/viewer/src/lib/notifications/NotificationOutlet.svelte
@@ -26,6 +26,11 @@
         },
         label: $t`Download`
       });
+    } else if (event.result == UpdateResult.Success) {
+      AppNotification.display(
+        $t`FieldWorks Lite has been updated successfully. Please restart the app to apply the changes.`,
+        {type: 'info', timeout: 'long'}
+      );
     }
   }, {includeLast: true});
 </script>
diff --git a/frontend/viewer/src/locales/en.po b/frontend/viewer/src/locales/en.po
@@ -457,6 +457,10 @@ msgstr "Field"
 msgid "Field Labels"
 msgstr "Field Labels"
 
+#: src/lib/notifications/NotificationOutlet.svelte
+msgid "FieldWorks Lite has been updated successfully. Please restart the app to apply the changes."
+msgstr "FieldWorks Lite has been updated successfully. Please restart the app to apply the changes."
+
 #: src/project/ProjectDropdown.svelte
 #: src/lib/OpenInFieldWorksButton.svelte
 #: src/home/HomeView.svelte
diff --git a/frontend/viewer/src/locales/es.po b/frontend/viewer/src/locales/es.po
@@ -462,6 +462,10 @@ msgstr "Campo"
 msgid "Field Labels"
 msgstr "Etiquetas de campo"
 
+#: src/lib/notifications/NotificationOutlet.svelte
+msgid "FieldWorks Lite has been updated successfully. Please restart the app to apply the changes."
+msgstr ""
+
 #: src/project/ProjectDropdown.svelte
 #: src/lib/OpenInFieldWorksButton.svelte
 #: src/home/HomeView.svelte
diff --git a/frontend/viewer/src/locales/fr.po b/frontend/viewer/src/locales/fr.po
@@ -462,6 +462,10 @@ msgstr "Champ d'application"
 msgid "Field Labels"
 msgstr "Étiquettes de champ"
 
+#: src/lib/notifications/NotificationOutlet.svelte
+msgid "FieldWorks Lite has been updated successfully. Please restart the app to apply the changes."
+msgstr ""
+
 #: src/project/ProjectDropdown.svelte
 #: src/lib/OpenInFieldWorksButton.svelte
 #: src/home/HomeView.svelte
diff --git a/frontend/viewer/src/locales/id.po b/frontend/viewer/src/locales/id.po
@@ -462,6 +462,10 @@ msgstr "Bidang"
 msgid "Field Labels"
 msgstr "Label Bidang"
 
+#: src/lib/notifications/NotificationOutlet.svelte
+msgid "FieldWorks Lite has been updated successfully. Please restart the app to apply the changes."
+msgstr ""
+
 #: src/project/ProjectDropdown.svelte
 #: src/lib/OpenInFieldWorksButton.svelte
 #: src/home/HomeView.svelte
diff --git a/frontend/viewer/src/locales/ko.po b/frontend/viewer/src/locales/ko.po
@@ -462,6 +462,10 @@ msgstr "필드"
 msgid "Field Labels"
 msgstr "필드 레이블"
 
+#: src/lib/notifications/NotificationOutlet.svelte
+msgid "FieldWorks Lite has been updated successfully. Please restart the app to apply the changes."
+msgstr ""
+
 #: src/project/ProjectDropdown.svelte
 #: src/lib/OpenInFieldWorksButton.svelte
 #: src/home/HomeView.svelte

Original file line number	Diff line number	Diff line change
`@@ -152,7 +152,7 @@ public DateTime LastUpdateCheck`
`152`	`152`	`set => preferences.Set(LastUpdateCheckKey, value);`
`153`	`153`	`}`
`154`	`154`
`155`		`- public bool SupportsAutoUpdate => true;`
	`155`	`+ public bool SupportsAutoUpdate => !FwLiteMauiKernel.IsPortableApp;`
`156`	`156`
`157`	`157`	`public bool IsOnMeteredConnection()`
`158`	`158`	`{`