limit_rate_after support variables

Ruslan Ermilov ru at nginx.com
Tue Feb 26 07:59:18 UTC 2019


On Mon, Feb 25, 2019 at 12:06:07PM +0100, Miroslav Nový wrote:
> Hello,
>   how does it look with merge into the main branch?

It's still under the review.  Sorry.

> Thank you for answer
> Mira Novy
> 
> po 10. 12. 2018 v 14:30 odesílatel Miroslav Novy <miranovy at gmail.com>
> napsal:
> 
> > Hello,
> >   your path is very good. I tested it and works fine for as.
> >
> > Reading complex value to the same local variable limit_rate was conflusing
> > for me at first time. But it is correct :)
> >
> >  if (r->limit_rate_after == 0
> > +            && clcf->limit_rate_after
> > +            && ngx_http_complex_value_size(r, clcf->limit_rate_after,
> > &val,
> > +                                           &limit_rate)
> > <-----------------------
> > +               == NGX_OK)
> > +        {
> > +            if (limit_rate != (size_t) NGX_ERROR) {
> > +                r->limit_rate_after = limit_rate;
> > +
> > +            } else if (val.len) {
> >
> > Please, merge your path to master branch.
> >
> > Best regards
> > M. Nový
> >


-- 
Ruslan Ermilov
Assume stupidity not malice


More information about the nginx-devel mailing list