Error: aborted in sanic 23.12

sanic-org / sanic

Accelerate your web app development | Build fast. Run fast.

https://sanic.dev

MIT License

17.95k stars 1.54k forks source link

Error: aborted in sanic 23.12 #2890

Open vincentdtt opened 7 months ago

vincentdtt commented 7 months ago

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug

When i call get method to get list data from server, with paging data (< 1000 object) sometime connection was lost and show Error: aborted but if i set paging > 1000 all request get Error: aborted. This error is gone when i downgrade to old version 23.6. i'm using haproxy for loading balance.

Code snippet

No response

Expected Behavior

No response

How do you run Sanic?

Sanic CLI

Operating System

Linux

Sanic Version

23.12

Additional context

No response

PushUpek commented 6 months ago

I have the same problem with Nginx reverse proxy. After day of checking commits from Sanic 23.12 changelog I found that this change #2831 is responsible for this problem. When I removed this method: https://github.com/sanic-org/sanic/blob/main/sanic/server/protocols/http_protocol.py#L231-L242 problem not occures any more.

ahopkins commented 6 months ago

Thanks for digging into this. Looking into it now .

Borsalinoo commented 3 months ago

I have the same problem with Nginx reverse proxy. After day of checking commits from Sanic 23.12 changelog I found that this change #2831 is responsible for this problem. When I removed this method: https://github.com/sanic-org/sanic/blob/main/sanic/server/protocols/http_protocol.py#L231-L242 problem not occures any more.

Great job, buddy! This issue has been troubling me for several days, very grateful for your answer!

ahopkins commented 1 month ago

Anyone have some good minimal reproducible cases to add to test suite?