fix: Fixes an issue where default storage was read-only

lib/storageUtil.js

@@ -22,7 +22,8 @@ var storageUtil = {};
 // https://connect.microsoft.com/IE/Feedback/Details/1496040
 storageUtil.browserHasLocalStorage = function() {
   try {
-    if (storageUtil.getLocalStorage()) {
+    var storage = storageUtil.getLocalStorage();
+    if (storageUtil.testStorage(storage)) {
       return true;
     } else {
       return false;
@@ -34,7 +35,8 @@ storageUtil.browserHasLocalStorage = function() {
 
 storageUtil.browserHasSessionStorage = function() {
   try {
-    if (storageUtil.getSessionStorage()) {
+    var storage = storageUtil.getSessionStorage();
+    if (storageUtil.testStorage(storage)) {
       return true;
     } else {
       return false;
@@ -73,4 +75,15 @@ storageUtil.getCookieStorage = function() {
   };
 };
 
+storageUtil.testStorage = function(storage) {
+  var key = 'okta-test-storage';
+  try {
+    storage.setItem(key, key);
+    storage.removeItem(key);
+    return true;
+  } catch (e) {
+    return false;
+  }
+};
+
 module.exports = storageUtil;

test/spec/tokenManager.js

@@ -44,6 +44,19 @@ describe('TokenManager', function() {
         'test-idToken': tokens.standardIdTokenParsed
       });
     });
+    it('defaults to sessionStorage if localStorage cannot be written to', function() {
+      jest.spyOn(window.console, 'log');
+      oauthUtil.mockStorageSetItemError();
+      var client = setupSync();
+      expect(window.console.log).toHaveBeenCalledWith(
+        '[okta-auth-sdk] WARN: This browser doesn\'t ' +
+        'support localStorage. Switching to sessionStorage.'
+      );
+      client.tokenManager.add('test-idToken', tokens.standardIdTokenParsed);
+      oauthUtil.expectTokenStorageToEqual(sessionStorage, {
+        'test-idToken': tokens.standardIdTokenParsed
+      });
+    });
     it('defaults to cookie-based storage if localStorage and sessionStorage are not available', function() {
       jest.spyOn(window.console, 'log');
       oauthUtil.mockLocalStorageError();
@@ -61,6 +74,27 @@ describe('TokenManager', function() {
         '2038-01-19T03:14:07.000Z'
       );
     });
+    it('defaults to cookie-based storage if sessionStorage cannot be written to', function() {
+      jest.spyOn(window.console, 'log');
+      oauthUtil.mockLocalStorageError();
+      oauthUtil.mockStorageSetItemError();
+      var client = setupSync({
+        tokenManager: {
+          storage: 'sessionStorage'
+        }
+      });
+      expect(window.console.log).toHaveBeenCalledWith(
+        '[okta-auth-sdk] WARN: This browser doesn\'t ' +
+        'support sessionStorage. Switching to cookie-based storage.'
+      );
+      var setCookieMock = util.mockSetCookie();
+      client.tokenManager.add('test-idToken', tokens.standardIdTokenParsed);
+      expect(setCookieMock).toHaveBeenCalledWith(
+        'okta-token-storage',
+        JSON.stringify({'test-idToken': tokens.standardIdTokenParsed}),
+        '2038-01-19T03:14:07.000Z'
+      );
+    });
   });
 
   describe('add', function() {

test/util/oauthUtil.js

@@ -29,6 +29,12 @@ oauthUtil.mockLocalStorageError = function() {
   });
 };
 
+oauthUtil.mockStorageSetItemError = function() {
+  jest.spyOn(Storage.prototype, 'setItem').mockImplementationOnce(function() {
+    throw 'This function is not supported on this system.';
+  });
+};
+
 oauthUtil.mockSessionStorageError = function() {
   jest.spyOn(storageUtil, 'getSessionStorage').mockImplementation(function() {
     throw 'This function is not supported on this system.';