unable to set_body_file on subrequest

david.birdsong · 2016-03-13T20:52:34+00:00

just found out this callback, https://github.com/openresty/lua-nginx-module#ngxon_abort, I think it will do the job.On Saturday, March 12, 2016 at 10:45...

unable to set_body_file on subrequest

david.birdsong

I'm pretty sure this should work, but I'm getting totally stuck in trying to convert a request body to a file located on the server.

The error log is not particularly helpful here:

2016/03/13 12:46:18 [error] 25108#0: *1 the request body was not read by ngx_lua, client: 127.0.0.1, server: , request: "GET /_repl?foo=bar HTTP/1.1", host: "127.0.0.1:8090"

I was able to pare down to a minimal config to reproduce the error:

https://gist.github.com/davidbirdsong/27d125c4b76faf63bdcf

agentzh

Hello!

On Sun, Mar 13, 2016 at 5:52 AM, David Birdsong wrote:
> I'm pretty sure this should work, but I'm getting totally stuck in trying to
> convert a request body to a file located on the server.
>
> The error log is not particularly helpful here:
>
> 2016/03/13 12:46:18 [error] 25108#0: *1 the request body was not read by
> ngx_lua, client: 127.0.0.1, server: , request: "GET /_repl?foo=bar
> HTTP/1.1", host: "127.0.0.1:8090"

The error message is clear, I think :) It's your responsibility to
read the request body (in the main request) before creating any
subrequests. In the NGINX core, subrequests can never read the request
bodies themselves and only main requests can.

Regards,
-agentzh

david.birdsong

On Wed, Mar 16, 2016 at 12:28 PM Yichun Zhang (agentzh) <age...@gmail.com> wrote:

Hello!

On Sun, Mar 13, 2016 at 5:52 AM, David Birdsong wrote:
> I'm pretty sure this should work, but I'm getting totally stuck in trying to
> convert a request body to a file located on the server.
>
> The error log is not particularly helpful here:
>
> 2016/03/13 12:46:18 [error] 25108#0: *1 the request body was not read by
> ngx_lua, client: 127.0.0.1, server: , request: "GET /_repl?foo=bar
> HTTP/1.1", host: "127.0.0.1:8090"

The error message is clear, I think :) It's your responsibility to
read the request body (in the main request) before creating any
subrequests. In the NGINX core, subrequests can never read the request
bodies themselves and only main requests can.

The request body is nil though. To be sure, I'd already tried discarding the request body before setting to a file with the same result.

I ended up using @pintsized's http library and wiring up a file_body_reader iterator function. It's less configuration, moves the logic to code which is easier to reason about.

Regards,
-agentzh.