implement $limit_rate_after, similar to $limit_rate

[Yuntaek LIM]

Hi,

This issue would be out of AMI-Support range. However, I would like to know if this is a bug before creating new ticket on 'nignx.org'.

When I set 'ri' and 'rs' parameters together, the value of 'rs' is not applied.
When client requests as an below example with a following configuration, 'limit_rate_after', which is dependent on 'rs' parameter,does not work.

Configuation

server {
        server_name test-domain.com
        root    /htdocs;

        location ~* \.flv {
            flv;
            if ($arg_LRA) {
                set $limit_rate_after $arg_LRA;
            }
            if ($arg_LR) {
                set $limit_rate "${arg_LR}k";
            }
        }
    }

Example of client request

curl -o /dev/null -H "Host: test-domain.com" http://localhost/test.flv?ri=100&rs=8192000

The result of client request shows that 'limit_rate' 100k has worked but limit_rate_after 8192000 has not worked.

I assumed that this is bug, so I've adjusted attached patch file which are modified below files.

./src/http/ngx_http_request.h
./src/http/ngx_http_variables.c
./src/http/ngx_http_write_filter_module.c

I would like to know if there is a problem in my configuration or it is bug.

[Ruslan Ermilov]

Could you please properly resubmit the patch?

[Yuntaek LIM]

I've just attached right file(limit_rate_after.3.patch). Sorry for submitting wrong file.

[Yuntaek Lim]

Replying to Ruslan Ermilov:

Could you please properly resubmit the patch?

I would like to know when you will consider this issue.

[Maxim Dounin]

The patch you attached is wrong as it overrides configuration, thus affecting subsequent request to the same worker process. Overral I think support for $limit_rate_after make sense, I'll take core of it.

[Maxim Dounin]

We've discussed this with Igor, and agreed that it's not a good idea to introduce more magic variables like $limit_rate. Instead, the "limit_rate" directive should be changed to support variables and the $limit_rate magic variable should be deprecated accordingly. Accordingly, there will be no $limit_rate_after, but the "limit_rate_after" directive will be changed to understand variables.

[Maxim Dounin]

Just for history, correct patch to implement $limit_rate_after.

[Ruslan Ermilov <ru@…>]

In 7504:c19ca381b2e6/nginx:

Variables support in limit_rate and limit_rate_after (ticket #293).