Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

QuotaExceededError on log database will bring up "Browser is missing features" #10891

Open
f0x52 opened this issue Sep 17, 2019 · 2 comments
Open

QuotaExceededError on log database will bring up "Browser is missing features" #10891

f0x52 opened this issue Sep 17, 2019 · 2 comments
Labels
A-Storage Storage layer of the app, including IndexedDB, local storage, etc. T-Defect

Comments

@f0x52
Copy link

f0x52 commented Sep 17, 2019

Upon accessing localStorage, Riot Web gets a "QuotaExceededError" (perhaps due to limited disk space). This is then wrongly caught as "Browser missing feature: 'localstorage'", which brings up the "Sorry, your browser is not able to run Riot."

Instead it would probably helpful to have a separate message for this specific case, as Riot will run fine in that browser when the storage problem is solved.

Happens in both riot.im/app and riot.im/develop on Firefox 70.0b6 (64-bit) (beta) on Linux
@f0x52 f0x52 added the T-Defect label Sep 17, 2019
@t3chguy
Copy link
Member

t3chguy commented Sep 18, 2019

Caused by https://github.com/Modernizr/Modernizr/blob/master/feature-detects/storage/localstorage.js#L43

@turt2live turt2live mentioned this issue Sep 24, 2019
Open
@jryans jryans added defect A-Storage Storage layer of the app, including IndexedDB, local storage, etc. labels Oct 28, 2019
@Ekleog
Copy link

Ekleog commented Nov 5, 2019

Looks like I got the same issue upon disk full

@turt2live turt2live mentioned this issue Apr 27, 2020
Closed
@jryans jryans removed the defect label Mar 4, 2021
@turt2live turt2live mentioned this issue Jul 8, 2022
Closed
@t3chguy t3chguy mentioned this issue Jan 16, 2023
Closed
@t3chguy t3chguy mentioned this issue Aug 24, 2023
Closed
@t3chguy t3chguy mentioned this issue Jun 17, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Storage Storage layer of the app, including IndexedDB, local storage, etc. T-Defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@jryans @Ekleog @t3chguy @f0x52