Merge pull request #7195 from mook-as/path-management/no-write-on-quit

background: Don't trigger settings update actions at shutdown
rancher-sandbox · Jul 16, 2024 · c5694ca · c5694ca
2 parents a4fe2d1 + b8a2854
commit c5694ca
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/background.ts b/background.ts
@@ -143,6 +143,12 @@ mainEvents.on('settings-update', async(newSettings) => {
   }
   k8smanager.debug = runInDebugMode;
 
+  if (gone) {
+    console.debug('Suppressing settings-update because app is quitting');
+
+    return;
+  }
+
   await setPathManager(newSettings.application.pathManagementStrategy);
   await pathManager.enforce();
 

diff --git a/pkg/rancher-desktop/utils/logging.ts b/pkg/rancher-desktop/utils/logging.ts
@@ -158,7 +158,7 @@ export class Log {
   }
 
   protected logWithDate(method: consoleKey, message: any, optionalParameters: any[]) {
-    this.console[method](`%s: ${ message }`, new Date(), ...optionalParameters);
+    this.console[method](`%s: ${ message }`, new Date().toISOString(), ...optionalParameters);
   }
 
   async sync() {