Make autoSync more robust for network errors

Catch uncaught error in autoSync that caused autoSync to stop syncing
Eugeny · Mar 23, 2024 · c906af5 · c906af5
1 parent 24c64a5
commit c906af5
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/tabby-settings/src/services/configSync.service.ts b/tabby-settings/src/services/configSync.service.ts
@@ -184,13 +184,17 @@ export class ConfigSyncService {
 
  private async autoSync () {
  while (true) {
- if (this.isEnabled() && this.config.store.configSync.auto) {
- const cfg = await this.getConfig(this.config.store.configSync.configID)
- if (new Date(cfg.modified_at) > this.lastRemoteChange) {
- this.logger.info('Remote config changed, downloading')
- this.download()
- this.lastRemoteChange = new Date(cfg.modified_at)
+ try {
+ if (this.isEnabled() && this.config.store.configSync.auto) {
+ const cfg = await this.getConfig(this.config.store.configSync.configID)
+ if (new Date(cfg.modified_at) > this.lastRemoteChange) {
+ this.logger.info('Remote config changed, downloading')
+ this.download()
+ this.lastRemoteChange = new Date(cfg.modified_at)
+ }
  }
+ } catch (error) {
+ this.logger.debug("Recovering from autoSync network error")
  }
  await new Promise(resolve => setTimeout(resolve, 60000))
  }