Add proper test for detecting unusable storage on iOS in Private Mode

Fixes #49
tus · Dec 3, 2016 · dfe32c8 · dfe32c8
1 parent 2889872
commit dfe32c8
Show file tree

Hide file tree

Showing 6 changed files with 29 additions and 16 deletions.
diff --git a/dist/tus.js b/dist/tus.js
diff --git a/dist/tus.js.map b/dist/tus.js.map
diff --git a/dist/tus.min.js b/dist/tus.min.js
diff --git a/dist/tus.min.js.map b/dist/tus.min.js.map
diff --git a/lib.es5/browser/storage.js b/lib.es5/browser/storage.js
diff --git a/lib/browser/storage.js b/lib/browser/storage.js
@@ -3,12 +3,17 @@
 let hasStorage = false;
 try {
   hasStorage = "localStorage" in window;
-  // Attempt to access localStorage
-  localStorage.length;
+
+  // Attempt to store and read entries from the local storage to detect Private
+  // Mode on Safari on iOS (see #49)
+  var key = "tusSupport";
+  localStorage.setItem(key, localStorage.getItem(key));
+
 } catch (e) {
   // If we try to access localStorage inside a sandboxed iframe, a SecurityError
-  // is thrown.
-  if (e.code === e.SECURITY_ERR) {
+  // is thrown. When in private mode on iOS Safari, a QuotaExceededError is
+  // thrown (see #49)
+  if (e.code === e.SECURITY_ERR || e.code === e.QUOTA_EXCEEDED_ERR) {
     hasStorage = false;
   } else {
     throw e;