Ngx_chunkin does not support PUT requests yet (Was Re: PUT support for chunked encoding with no Cont

Server NGINX
1

On Mon, Apr 26, 2010 at 12:35 AM, Aaron N. [email protected]
wrote:

On Sun, Apr 25, 2010 at 4:06 AM, agentzh [email protected] wrote:

You may find the ngx_chunkin module helpful:

http://wiki.nginx.org/NginxHttpChunkinModule

Yep, I tried this module right off the bat and received the same
error.

Okay, my bad. ngx_chunkin does not support PUT requests yet (The nginx
core goes another route for non-POST requests with chunked encoding.).

I’ve never tried PUT in my test suite. I’ll try fixing it in the next
release. It should be a simple fix in the
ngx_http_chunkin_is_chunked_encod
ing function. Thanks for the report!

BTW, nginx will still print that error message to error.log even if
ngx_chunkin works. Because nginx aggressively checks this case when it
parses request headers and ngx_chunkin cannot take any action before
that. Commenting out that ngx_log_error line in the nginx core should
work :wink:

Cheers,
-agentzh

2

On Tue, Apr 27, 2010 at 10:23 PM, agentzh [email protected] wrote:

I’ve never tried PUT in my test suite. I’ll try fixing it in the next
release. It should be a simple fix in the
ngx_http_chunkin_is_chunked_encod
ing function. Thanks for the report!

Great - thanks for the update. I’m anxious to try this out and am
happy to test out builds once you have something you think is ready
for release. Feel free to contact me directly if I can help.

Thanks,
Aaron

3

On Tue, Apr 27, 2010 at 10:23 PM, agentzh [email protected] wrote:

Okay, my bad. ngx_chunkin does not support PUT requests yet (The nginx
core goes another route for non-POST requests with chunked encoding.).

I’ve never tried PUT in my test suite. I’ll try fixing it in the next
release. It should be a simple fix in the
ngx_http_chunkin_is_chunked_encod
ing function. Thanks for the report!

Did this ever make it into a release? I’d love to try it out.

Thanks,
Aaron

4

On Fri, Jul 23, 2010 at 2:15 AM, Aaron N. [email protected]
wrote:

Just for the record, ngx_chunkin has now got chunked PUT support since
the v0.20 release:

http://wiki.nginx.org/NginxHttpChunkinModule#v0.20

Have fun!
-agentzh

5

On Fri, Jul 23, 2010 at 2:15 AM, Aaron N. [email protected]
wrote:

Did this ever make it into a release? I’d love to try it out.

Not yet. Been busy with many other stuffs. I’ll let you know when I’m on
it.

Cheers,
-agentzh

6

On Mon, Aug 2, 2010 at 1:48 AM, agentzh [email protected] wrote:

Just for the record, ngx_chunkin has now got chunked PUT support since
the v0.20 release:

Changelog says support for chunked POST support - is PUT support
included in that? I thought POST support was already there when this
thread was going on.

7

On Wed, Aug 4, 2010 at 12:55 AM, Aaron N. [email protected]
wrote:

On Mon, Aug 2, 2010 at 1:48 AM, agentzh [email protected] wrote:

Just for the record, ngx_chunkin has now got chunked PUT support since
the v0.20 release:

Changelog says support for chunked POST support - is PUT support
included in that? I thought POST support was already there when this
thread was going on.

Nice catch :slight_smile: It was a typo and I’ve fixed it on the wiki page :stuck_out_tongue:

Cheers,
-agentzh