Dexie.js: SecurityError when storage is disabled (again)

[asimen1]

Uncaught DOMException: Failed to read the 'localStorage' property from 'Window': Access is denied for this document.

This was previously fixed in https://github.com/dfahlander/Dexie.js/issues/158 (v1.3.4)

But I am getting this error again (on Chrome 62 when disabling cookies in privacy settings).
Seems like this commit removed the try&catch when attempting to access localStorage.

Can be easily reproduced with latest Chrome (62) + fiddle + disabling cookies.

(Using latest Dexie release 2.0.1)

[dfahlander]

Dexie has removed it's dependency of localStorage so it can't be Dexie that complains.

[dfahlander]

I ran the fiddle with cookies disabled and it seems Google Analytics code on jsfiddle was the first issue, and then some other libs that was complaining too.

[asimen1]

Maybe fiddle wasn't the best idea for testing this since there are additional errors like you noticed.
However, I clearly see the error from Dexie aswell...

Developer tools image:

This is the line in the code where checks localStorage and causes the error: https://github.com/dfahlander/Dexie.js/blob/ffa52d1ad5b39b8316760a50459d4779a6171862/src/Dexie.js#L3187

[dfahlander]

I see! Forgot about that one ;)

[gregor]

@dfahlander Can this issue get some love with a fix even on the 2.x branch. The Wire app just shows a blank page because of it instead of our cookie warning page. It would be great to get this fixed in order to help our users and decrease the volume of related request to our CS team.

[asimen1]

+1
I would also love to see this fixed in 2.x.

[dfahlander]

Released fix in 2.0.3.

[gregor]

Thanks for fixing this. Will get this fixed in https://github.com/wireapp/wire-webapp in the morning.