]> git.kaiwu.me - nginx.git/commitdiff
Xslt: disabled ranges.
authorRoman Arutyunyan <arut@nginx.com>
Wed, 22 Jul 2020 19:16:19 +0000 (22:16 +0300)
committerRoman Arutyunyan <arut@nginx.com>
Wed, 22 Jul 2020 19:16:19 +0000 (22:16 +0300)
Previously, the document generated by the xslt filter was always fully sent
to client even if a range was requested and response status was 206 with
appropriate Content-Range.

The xslt module is unable to serve a range because of suspending the header
filter chain.  By the moment full response xml is buffered by the xslt filter,
range header filter is not called yet, but the range body filter has already
been called and did nothing.

The fix is to disable ranges by resetting the r->allow_ranges flag much like
the image filter that employs a similar technique.

src/http/modules/ngx_http_xslt_filter_module.c

index b2f107dc0ac89e44cc7a75bbfede43f07b3aebc4..8afd656af444f2a5840abbad31c7670e53c2bfc6 100644 (file)
@@ -233,6 +233,7 @@ ngx_http_xslt_header_filter(ngx_http_request_t *r)
     ngx_http_set_ctx(r, ctx, ngx_http_xslt_filter_module);
 
     r->main_filter_need_in_memory = 1;
+    r->allow_ranges = 0;
 
     return NGX_OK;
 }