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.

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

If the browser capacity is full, it occurs Quota limit Exceeded error #50

Open
ohute-couture opened this issue Oct 1, 2021 · 6 comments
Labels

Comments

@ohute-couture
Copy link

it need to add an error handler
please confirm this error

@daybrush
Copy link
Member

daybrush commented Oct 8, 2021

@ohute-couture

Hi. I did some research on the code.

I have a few questions

  1. Is the data you want to store very large?
  2. Can you check the state__persist__ of storage before the error occurs? You may send it to me through WORKS.

@ohute-couture
Copy link
Author

@daybrush now I get it

  1. The data we want to store is not large, but depending on the user environment, the browser capacity seems to be small in some cases.
  2. This error occurs when egjs-persist tries to save the key value. It happens on line 144 of the code attached in issue

@daybrush
Copy link
Member

daybrush commented Feb 14, 2022

@ohute-couture

Test version 2.5.1.

I have some guesses about the problem.

  1. The stored value for persist is large.
  2. The values of other previously stored items are large.

So, I want to check the error as follows.
https://github.com/naver/egjs-persist#handle-the-error

@ohute-couture
Copy link
Author

I checked error when the capacity is exceeded.

There is an error when loading SSR . Please check.
This code seems to be cause.
https://github.com/naver/egjs-persist/pull/51/files#diff-13e1bbff6f8b004d7f66ddfd1574d7133cad5ed2c9143a9c8d7177715ef9bfb8R34

@daybrush
Copy link
Member

@ohute-couture

persist 2.5.2 is released. Check it again.

@ohute-couture
Copy link
Author

I confirmed that it works
Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants