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

Message length exceeded maximum allowed length #543

Closed
mbtts opened this issue Aug 1, 2018 · 4 comments
Closed

Message length exceeded maximum allowed length #543

mbtts opened this issue Aug 1, 2018 · 4 comments

Comments

@mbtts
Copy link

mbtts commented Aug 1, 2018

I am seeing this error (“Message length exceeded maximum allowed length”) when the panel is initialising in Chrome.

There are similar issues posted for other plugins:

message length exceeded maximum allowed length

It does work in Firefox.

Chrome version: 68.0.3440.75
Redux Dev Tools version: 2.15.3

@ghost
Copy link

ghost commented Aug 15, 2018

me too

@mbtts
Copy link
Author

mbtts commented Aug 18, 2018

If possible the workaround is to try and reduce the size of your store, either by normalizing it or only storing a ref/key in the store and keeping the large objects in another state variable outside of redux.

The second is more dramatic and maybe not even possible depending on your use case.

I guess the long term fix is to do what the spector author did and split the messages up into chunks and re-assemble on the other side.

@zalmoxisus
Copy link
Owner

Thanks for the details and offered solutions! We could go with splitting the messages in future, but there's a reason why Chrome imposed this limit in last versions. Having large states will make the extension slow and will freeze/crash at some point. So not sure we should support that.

I added a warning with the link on how to use sanitizers to fix that in case the message has more than 16 MB. Open for discussions if someone still thinks we should support that.

@zalmoxisus
Copy link
Owner

I added splitting messages in #582.2.16 should land in a day on Chrome Store. Thanks again for investigating this!

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

No branches or pull requests

2 participants