Clean up localStorage after testing its availability

tus · Dec 5, 2022 · 4f3bd09 · YEENSLy · Dec 16, 2022 · 4f3bd09
1 parent bccee72
commit 4f3bd09
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/lib/browser/urlStorage.js b/lib/browser/urlStorage.js
@@ -4,8 +4,12 @@ try {
 
   // Attempt to store and read entries from the local storage to detect Private
   // Mode on Safari on iOS (see #49)
+  // If the key was not used before, we remove it from local storage again to
+  // not cause confusion where the entry came from.
   const key = 'tusSupport'
-  localStorage.setItem(key, localStorage.getItem(key))
+  const originalValue = localStorage.getItem(key)
+  localStorage.setItem(key, originalValue)
+  if (originalValue === null) localStorage.removeItem(key)
 } catch (e) {
   // If we try to access localStorage inside a sandboxed iframe, a SecurityError
   // is thrown. When in private mode on iOS Safari, a QuotaExceededError is