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

Put in place content length checks to make problems more visible #243

Open
nickwilliams-eventbrite opened this issue Feb 4, 2020 · 0 comments
Assignees
Milestone

Comments

@nickwilliams-eventbrite
Copy link
Contributor

nickwilliams-eventbrite commented Feb 4, 2020

Issues with chunked message reassembly in the Redis transport, such as #240, would be easier to diagnose if we included chunk-length and content-length headers and performed sanity checks on received chunks and re-assembled messages with those headers. However, such a change cannot be included in the fix for #240, because it will require a new minor version and a new Redis protocol version (4).

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

No branches or pull requests

1 participant