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

ValueError: Too many packets in payload #77

Closed
ftsell opened this issue Jul 3, 2020 · 2 comments
Closed

ValueError: Too many packets in payload #77

ftsell opened this issue Jul 3, 2020 · 2 comments
Assignees
Labels
backend bug Something isn't working

Comments

@ftsell
Copy link
Contributor

ftsell commented Jul 3, 2020

Sentry - Please ask Timon or me for further details.

ValueError: Too many packets in payload
  File "engineio/server.py", line 396, in handle_request
    socket.handle_post_request(environ)
  File "engineio/socket.py", line 128, in handle_post_request
    p = payload.Payload(encoded_payload=body)
  File "engineio/payload.py", line 15, in __init__
    self.decode(encoded_payload)
  File "engineio/payload.py", line 76, in decode
    raise ValueError('Too many packets in payload')
@ftsell ftsell added bug Something isn't working backend labels Jul 3, 2020
@Flova
Copy link
Member

Flova commented Jul 3, 2020

miguelgrinberg/python-engineio#142

Maybe this is related.

@Flova
Copy link
Member

Flova commented Jul 19, 2020

Thx for @timonegk for the additional info.

The error seems to be not relevant in a realistic use-case because socket.io throws an error if more than 50 state updates are sent to the user via one http request as described in the linked issue. This happens due to security reasons.
In the described case ~120 updates are packed in one request, this is only possible if the user triggers many state updates in a short time (hundreds per second). The user could e.g. fall asleep on the spacebar or something like that.

This is only reasoned form the sentry reporting and could not be reproduced in any way.

@Flova Flova closed this as completed Jul 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants