bitwarden/browser
1
0
Fork 0
mirror of https://github.com/bitwarden/browser synced 2025-12-16 00:03:56 +00:00
Code Issues Releases Wiki Activity

store in-flight sync so multiple calls to the sync service are avoided

Browse Source
This commit is contained in:
Nick Krantz
2025-05-12 11:34:03 -05:00
parent 3f686ac6a4
commit 233c8e9d4b
2 changed files with 58 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
libs/common/src/platform/sync/default-sync.service.ts
View File

@@ -59,6 +59,9 @@ import { SyncOptions } from "./sync.service";
export class DefaultSyncService extends CoreSyncService {
syncInProgress = false;
/** The promise associated with any in-flight sync operations. When null, no sync is in-flight. */
private inFlightSync: Promise<boolean> | null = null;
constructor(
private masterPasswordService: InternalMasterPasswordServiceAbstraction,
accountService: AccountService,
@@ -107,7 +110,16 @@ export class DefaultSyncService extends CoreSyncService {
forceSync: boolean,
allowThrowOnErrorOrOptions?: boolean | SyncOptions,
): Promise<boolean> {
return this._fullSync(forceSync, allowThrowOnErrorOrOptions);
if (this.inFlightSync !== null) {
return this.inFlightSync;
}
this.inFlightSync = this._fullSync(forceSync, allowThrowOnErrorOrOptions).finally(() => {
// Reset the in-flight sync promise when it completes
this.inFlightSync = null;
});
return this.inFlightSync;
}
private async needsSyncing(forceSync: boolean) {