Fix isSafari() throwing on React Native (fixes #7962) (#7963)

In #7929 some logic was added to Firestore to check for the browser being Safari. This new check for isSafari() unexpectedly threw an exception in React Native. This PR fixes the exception by explicitly checking for an object not being undefined before using it. Fixes #7962
firebase · Jan 19, 2024 · 434f841 · 434f841
1 parent 4b5a82e
commit 434f841
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/.changeset/purple-cooks-explode.md b/.changeset/purple-cooks-explode.md
@@ -0,0 +1,5 @@
+---
+'@firebase/util': patch
+---
+
+Fix isSafari() throwing on React Native
diff --git a/packages/util/src/environment.ts b/packages/util/src/environment.ts
@@ -138,6 +138,7 @@ export function isNodeSdk(): boolean {
 export function isSafari(): boolean {
   return (
     !isNode() &&
+    !!navigator.userAgent &&
     navigator.userAgent.includes('Safari') &&
     !navigator.userAgent.includes('Chrome')
   );