You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Aug 30, 2023. It is now read-only.
Uncaught DOMException: Failed to read the 'sessionStorage' property from 'Window':
Access is denied for this document.
Expected Result
Error is caught and handled.
Actual Result
Error is not caught, can tank applications depending on how they're set up.
This appears to be because access to sessionStorage was moved out of a try/catch block in fetchSession as part of #271
The fix is probably to move access back inside the try/catch block.
To avoid needing to keep on top of safe session storage access everywhere, I'd also consider having a central getSafelyFromSessionStorage function, and add a linting rule to prevent calls to window.sessionStorage etc everywhere else.