Revert "refactor(ext/http): rewrite hyper integration and fix bug (#12332)"

[lucacasonato]

This reverts commit 5b1e537.

This change updates some behaviour that caused std/http to break. We will try to reland this for 1.16.1.

[bartlomieju]

RSLGTM

[piscisaureus]

Any background on exactly how this was breaking std/http?

[lucacasonato]

Yes, the serve should handle websocket requests test in std/http/server_test.ts fails. Root cause seemed to be that previously Deno.closeing a Deno.HttpConn would not close websockets created by Deno.upgradeWebsocket, but now it does.

[piscisaureus]

Yes, the serve should handle websocket requests test in std/http/server_test.ts fails.

Thanks for the pointer!

Root cause seemed to be that previously Deno.closeing a Deno.HttpConn would not close websockets created by Deno.upgradeWebsocket, but now it does.

That's not the root cause; closing HttpConn doesn't close websockets before nor after the patch.
It seems that this test is hanging because the TcpListener created by serve() stays open somehow.

[lucacasonato]

@piscisaureus If you add some onerror handlers to the socket, you will see that the WebSocket never gets established.