Skip to content

Commit

Permalink
Move trust logic to only occur when fingerprint setting is enabled
Browse files Browse the repository at this point in the history
  • Loading branch information
quexten committed Jan 20, 2025
1 parent 946a23f commit af1025f
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -444,11 +444,11 @@ export class BiometricMessageHandlerService {
}

async validateFingerprint(appId: string): Promise<boolean> {
if ((await this.connectedApps.has(appId)) && (await this.connectedApps.get(appId)).trusted) {
return true;
}

if (await firstValueFrom(this.desktopSettingService.browserIntegrationFingerprintEnabled$)) {
if ((await this.connectedApps.has(appId)) && (await this.connectedApps.get(appId)).trusted) {
return true;

Check warning on line 449 in apps/desktop/src/services/biometric-message-handler.service.ts

View check run for this annotation

Codecov / codecov/patch

apps/desktop/src/services/biometric-message-handler.service.ts#L449

Added line #L449 was not covered by tests
}

ipc.platform.nativeMessaging.sendMessage({

Check warning on line 452 in apps/desktop/src/services/biometric-message-handler.service.ts

View check run for this annotation

Codecov / codecov/patch

apps/desktop/src/services/biometric-message-handler.service.ts#L452

Added line #L452 was not covered by tests
command: "verifyDesktopIPCFingerprint",
appId: appId,
Expand Down

0 comments on commit af1025f

Please sign in to comment.