getNotificationPermissions ignore mobile ios

fingerprintjs · Nov 26, 2023 · 0f0dbb8 · 0f0dbb8
1 parent be7cfbf
commit 0f0dbb8
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/playground/index.ts b/playground/index.ts
@@ -6,6 +6,7 @@ import {
   getMozAppearanceSupport,
   isAndroid,
   isDesktopWebKit,
+  isIPad,
 } from '../src/utils/browser'
 import './style.css'
 
@@ -44,6 +45,7 @@ const runDetection = async (): Promise<DetectionResult> => {
       mozAppearanceSupport: getMozAppearanceSupport(),
       isAndroid: isAndroid(),
       isDesktopWebKit: isDesktopWebKit(),
+      isIPad: isIPad(),
     }
 
     return {

diff --git a/src/sources/notification_permissions.ts b/src/sources/notification_permissions.ts
@@ -1,7 +1,12 @@
-import { BotdError, State } from '../types'
+import { BotdError, BrowserEngineKind, State } from '../types'
+import { getBrowserEngineKind, isDesktopWebKit } from '../utils/browser'
 
 export default async function getNotificationPermissions(): Promise<boolean> {
   if (window.Notification === undefined) {
+    // if on mobile ios, ignore
+    if (getBrowserEngineKind() === BrowserEngineKind.Webkit && !isDesktopWebKit()) {
+      return false
+    }
     throw new BotdError(State.Undefined, 'window.Notification is undefined')
   }
   if (navigator.permissions === undefined) {