diff options
author | Loïc Hoguin <[email protected]> | 2014-06-02 23:09:43 +0200 |
---|---|---|
committer | Loïc Hoguin <[email protected]> | 2014-06-02 23:09:43 +0200 |
commit | 0c379256423f9db3b1a280b686baa0b965a4c739 (patch) | |
tree | e40a3425d9cea40eb3577239a66c7ad5f916b886 /test/http_SUITE.erl | |
parent | 1ad3aae4d5459ee991805d16a3837e1da2ba96ff (diff) | |
download | cowboy-0c379256423f9db3b1a280b686baa0b965a4c739.tar.gz cowboy-0c379256423f9db3b1a280b686baa0b965a4c739.tar.bz2 cowboy-0c379256423f9db3b1a280b686baa0b965a4c739.zip |
Add request body reading options
The options were added to allow developers to fix timeout
issues when reading large bodies. It is also a cleaner and
easier to extend interface.
This commit deprecates the functions init_stream, stream_body
and skip_body which are no longer needed. They will be removed
in 1.0.
The body function can now take an additional argument that is a
list of options. The body_qs, part and part_body functions can
too and simply pass this argument down to the body call.
There are options for disabling the automatic continue reply,
setting a maximum length to be returned (soft limit), setting
the read length and read timeout, and setting the transfer and
content decode functions.
The return value of the body and body_qs have changed slightly.
The body function now works similarly to the part_body function,
in that it returns either an ok or a more tuple depending on
whether there is additional data to be read. The body_qs function
can return a badlength tuple if the body is too big. The default
size has been increased from 16KB to 64KB.
The default read length and timeout have been tweaked and vary
depending on the function called.
The body function will now adequately process chunked bodies,
which means that the body_qs function will too. But this means
that the behavior has changed slightly and your code should be
tested properly when updating your code.
The body and body_qs still accept a length as first argument
for compatibility purpose with older code. Note that this form
is deprecated and will be removed in 1.0. The part and part_body
function, being new and never having been in a release yet, have
this form completely removed in this commit.
Again, while most code should work as-is, you should make sure
that it actually does before pushing this to production.
Diffstat (limited to 'test/http_SUITE.erl')
-rw-r--r-- | test/http_SUITE.erl | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/http_SUITE.erl b/test/http_SUITE.erl index 35d4b06..8ab99cb 100644 --- a/test/http_SUITE.erl +++ b/test/http_SUITE.erl @@ -336,7 +336,7 @@ echo_body(Config) -> %% Check if sending request whose size is bigger than 1000000 bytes causes 413 echo_body_max_length(Config) -> ConnPid = gun_open(Config), - Ref = gun:post(ConnPid, "/echo/body", [], << 0:8000008 >>), + Ref = gun:post(ConnPid, "/echo/body", [], << 0:2000000/unit:8 >>), {response, nofin, 413, _} = gun:await(ConnPid, Ref), ok. @@ -350,7 +350,7 @@ echo_body_qs(Config) -> echo_body_qs_max_length(Config) -> ConnPid = gun_open(Config), - Ref = gun:post(ConnPid, "/echo/body_qs", [], << "echo=", 0:15996/unit:8 >>), + Ref = gun:post(ConnPid, "/echo/body_qs", [], << "echo=", 0:2000000/unit:8 >>), {response, nofin, 413, _} = gun:await(ConnPid, Ref), ok. |