clj-commons / aleph

Asynchronous streaming communication for Clojure - web server, web client, and raw TCP/UDP
http://aleph.io
MIT License
2.54k stars 241 forks source link

Support more options when creating HttpServerCodec #660

Closed arnaudgeiser closed 1 year ago

arnaudgeiser commented 1 year ago

Description

Support three of the four missing options:

but not allowPartialChunks which is pretty hard to document and not an obvious use case : https://github.com/netty/netty/pull/11469

Here are the default values : https://github.com/netty/netty/blob/4.1/codec-http/src/main/java/io/netty/handler/codec/http/HttpObjectDecoder.java#L132-L139

Close #448

arnaudgeiser commented 1 year ago

The tests are unstable due to the time it takes for some servers (TCP/UDP) to be shutdown. I will merge this PR as is and create another PR to stabilize the tests.