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

Receiving many large data results in disconnect #3184

Closed
moo1 opened this issue Jul 16, 2024 · 2 comments
Closed

Receiving many large data results in disconnect #3184

moo1 opened this issue Jul 16, 2024 · 2 comments

Comments

@moo1
Copy link

moo1 commented Jul 16, 2024

I encountered unexpected disconnect when receiving several large data simultaneously (8+ 100-300KB messages). Debugging this and found the disconnect was initiated in ops-ws.c rops_handle_POLLIN_ws() where sanity went more than 10. I modified the initial sanity to be larger like 100 and no issue in receiving those large data.

My questions are;

  • Why the limit so low? Is intended usage dealing with smaller data?
  • Are there any config to avoid this issue or modifying code to increase the initial sanity only solution?
  • Why it treats as disconnect instead of processing remaining data in the next polling?
@davehorton
Copy link

davehorton commented Jul 18, 2024

hi @moo1 I just realized my issue #3185 is the same cause as yours :)

@lws-team
Copy link
Member

I pushed a patch increasing sanity to 100.

The reason this exists is that depending on your platform, it's possible for the tls library to lose the underlying socket connection and act badly, eg, spin forever delivering 0 length reads without error.

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

3 participants