From e940a96d2a9347327e2d8cba234ce8ccd89e0cf2 Mon Sep 17 00:00:00 2001 From: Alex Date: Fri, 9 Jan 2026 15:21:43 -0500 Subject: [PATCH] return null on checksum match --- .../phishing-detection/services/phishing-data.service.ts | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/apps/browser/src/dirt/phishing-detection/services/phishing-data.service.ts b/apps/browser/src/dirt/phishing-detection/services/phishing-data.service.ts index 21fe74f1873..e5b4c6fd45f 100644 --- a/apps/browser/src/dirt/phishing-detection/services/phishing-data.service.ts +++ b/apps/browser/src/dirt/phishing-detection/services/phishing-data.service.ts @@ -152,13 +152,11 @@ export class PhishingDataService { const applicationVersion = await this.platformUtilsService.getApplicationVersion(); - // If checksum matches, return existing data with new timestamp & version + // If checksum matches, skip state write to avoid expensive Set rebuild const remoteChecksum = await this.fetchPhishingChecksum(this.resourceType); if (remoteChecksum && prev.checksum === remoteChecksum) { - this.logService.info( - `[PhishingDataService] Remote checksum matches local checksum, updating timestamp only.`, - ); - return { ...prev, timestamp, applicationVersion }; + this.logService.info(`[PhishingDataService] Checksum match, skipping update.`); + return null; } // Checksum is different, data needs to be updated.