andras/vault-link
1
0
Fork 0
Code Issues Pull requests 18 Projects Releases 55 Packages Wiki Activity Actions 1

Add lock on settings

Browse source
This commit is contained in:
Andras Schmelczer 2025-11-30 14:42:50 +00:00
parent 3517af1461
commit 39860f7f04
1 changed files with 20 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

36
frontend/sync-client/src/persistence/settings.ts
View file

@ -1,5 +1,6 @@
import type { Logger } from "../tracing/logger";
import { awaitAll } from "../utils/await-all";
import { Lock } from "../utils/data-structures/locks";
export interface SyncSettings {
remoteUri: string;
@ -33,6 +34,7 @@ export const DEFAULT_SETTINGS: SyncSettings = {
export class Settings {
private settings: SyncSettings;
private readonly lock: Lock = new Lock();
private readonly onSettingsChangeHandlers: ((
newSettings: SyncSettings,
@ -83,24 +85,26 @@ export class Settings {
}
public async setSettings(value: Partial<SyncSettings>): Promise<void> {
this.logger.debug(`Updating settings with: ${JSON.stringify(value)}`);
const oldSettings = this.settings;
this.settings = {
...this.settings,
...value
};
await this.lock.withLock(async () => {
this.logger.debug(`Updating settings with: ${JSON.stringify(value)}`);
const oldSettings = this.settings;
this.settings = {
...this.settings,
...value
};
await awaitAll(
this.onSettingsChangeHandlers
.map((handler) => {
return handler(this.settings, oldSettings);
})
.filter((result): result is Promise<unknown> => {
return result instanceof Promise;
})
);
await awaitAll(
this.onSettingsChangeHandlers
.map((handler) => {
return handler(this.settings, oldSettings);
})
.filter((result): result is Promise<unknown> => {
return result instanceof Promise;
})
);
await this.save();
await this.save();
});
}
private async save(): Promise<void> {