diff --git a/apps/browser/src/dirt/phishing-detection/services/phishing-detection.service.spec.ts b/apps/browser/src/dirt/phishing-detection/services/phishing-detection.service.spec.ts index ceb18bd1573..06a37f12faa 100644 --- a/apps/browser/src/dirt/phishing-detection/services/phishing-detection.service.spec.ts +++ b/apps/browser/src/dirt/phishing-detection/services/phishing-detection.service.spec.ts @@ -79,4 +79,9 @@ describe("PhishingDetectionService", () => { // phishingDetectionSettingsService, // ); // }); + + // TODO + // it("should not enable phishing detection for safari", () => { + // + // }); }); diff --git a/apps/browser/src/dirt/phishing-detection/services/phishing-detection.service.ts b/apps/browser/src/dirt/phishing-detection/services/phishing-detection.service.ts index e04d08559ab..501dfbf7a50 100644 --- a/apps/browser/src/dirt/phishing-detection/services/phishing-detection.service.ts +++ b/apps/browser/src/dirt/phishing-detection/services/phishing-detection.service.ts @@ -5,6 +5,7 @@ import { filter, map, merge, + of, Subject, switchMap, tap, @@ -111,7 +112,17 @@ export class PhishingDetectionService { .messages$(PHISHING_DETECTION_CANCEL_COMMAND) .pipe(switchMap((message) => BrowserApi.closeTab(message.tabId))); - const phishingDetectionActive$ = phishingDetectionSettingsService.on$; + // Phishing detection is unavailable on Safari due to platform limitations + if (BrowserApi.isSafariApi) { + logService.debug( + "[PhishingDetectionService] Disabling phishing detection service for Safari.", + ); + } + + // Watching for settings changes to enable/disable phishing detection + const phishingDetectionActive$ = BrowserApi.isSafariApi + ? of(false) + : phishingDetectionSettingsService.on$; const initSub = phishingDetectionActive$ .pipe(