Reproducible: Server hangs on every other request

[m0rt3nlund]

Reproducible: Server hangs on every other request

[m0rt3nlund]

Thank you very much for a prompt and good answer! This project is great!

I of course should have returned the new conn, now everything works as expected!

My thought was that since the docs says that the body is read out of the socket I did not think there was anything in the conn that was updated and then did not even think that the new conn was needed.. Silly me..

But then I know a bunch more about the inner workings of both Plug and Bandit

Thanks!

[m0rt3nlund]

[mtrudel]

Are you 100% certain you're not rebinding conn on every call into Plug.Conn functions? This looks exactly like that's the case

[mtrudel]

Here's your problem:

https://github.com/m0rt3nlund/bandit_test/blob/main/lib/teste_web/controllers/page_controller.ex#L38

You need to rebind conn on every call - Bandit keeps important things in there

[mtrudel]

Cowboy gets away with this in some cases because they smear connection state across a few processes so you won't always see this (but you will sometimes; it depends on where you're not rebinding). Bandit is very much intentionally a single-process-per-connection design, so we're more sensitive to rebinding.

Regardless, the Plug contract is clear on this; you need to track and rebind returned conn values for correctness.

[mtrudel]

Closing for hygiene. Thanks for the report!