Merge pull request #75 from iam4x/master

Catch `QuotaExceededError`
remix-run · Oct 7, 2015 · 64af5ea · 64af5ea
2 parents aeb8ec8 + c54f8ef
commit 64af5ea
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/.gitignore b/.gitignore
@@ -1,2 +1,3 @@
 lib
 umd
+node_modules
diff --git a/modules/DOMStateStorage.js b/modules/DOMStateStorage.js
@@ -1,3 +1,5 @@
+import warning from 'warning'
+
 /*eslint-disable no-empty */
 const KeyPrefix = '@@History/'
 
@@ -6,7 +8,14 @@ function createKey(key) {
 }
 
 export function saveState(key, state) {
-  window.sessionStorage.setItem(createKey(key), JSON.stringify(state))
+  try {
+    window.sessionStorage.setItem(createKey(key), JSON.stringify(state))
+  } catch (error) {
+    if (error.name === 'QuotaExceededError')
+      return warning(null, 'sessionStore is not accessible in incognito mode')
+
+    throw error
+  }
 }
 
 export function readState(key) {