CrowdHailer / Ace

HTTP web server and client, supports http1 and http2

Home Page:https://hex.pm/packages/ace

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Large response body fails

varnerac opened this issue · comments

When returning a response over 65k bytes, Ace's HTTP2 service fails, with a timeout. We've verified locally with Mojito and Gun clients. I've added a reproducer here:

https://github.com/varnerac/Ace/tree/large_response_reproducer/

Could you open a PR to add this test, even without the fix.

However instead of modifying simple app just make the greeting be an enormous binary https://github.com/varnerac/Ace/blob/large_response_reproducer/test/ace/http2_test.exs#L280

Hi @varnerac could you eventually solve the large body issue?

I seem to be having same/similar issues downloading large files via ssl, same code worked fine on clear-text

I did not

@varnerac Please see if this works for you #154

@CharlesOkwuagwu I’ll try to look this evening. In the meantime, you could add the test from #151 to your PR as a test.