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

Limit queue sizes #122

Open
talex5 opened this issue Sep 9, 2017 · 0 comments
Open

Limit queue sizes #122

talex5 opened this issue Sep 9, 2017 · 0 comments

Comments

@talex5
Copy link
Contributor

talex5 commented Sep 9, 2017

Currently there is no bound on the size of the queue in Endpoint. A client that keeps sending requests and never reads responses will keep using up more and more memory in the server.

We should add some stress tests and try to handle this. Dropping individual messages could break E-order, so we should probably drop the entire connection. We could also try not reading more input until the output has been consumed, although on its own this could deadlock.

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

1 participant