Hello Yann, We can confirm this patch works on our end. We’ll apply this and send out an update.
> On Jul 19, 2018, at 2:41 PM, Yann Ylavic <ylavic....@gmail.com> wrote:
>
> On Thu, Jul 19, 2018 at 8:23 PM, Luca Toscano <toscano.l...@gmail.com> wrote:
>>
>> Yann, any idea?
>
> Looks like we missed the simplest case :/
>
> Index: modules/filters/mod_ratelimit.c
> ===================================================================
> --- modules/filters/mod_ratelimit.c (revision 1835556)
> +++ modules/filters/mod_ratelimit.c (working copy)
> @@ -208,7 +208,7 @@ rate_limit_filter(ap_filter_t *f, apr_bucket_briga
> ap_remove_output_filter(f);
> }
> else if (!APR_BUCKET_IS_FLUSH(e)) {
> - if (APR_BRIGADE_EMPTY(bb)) {
> + if (ctx->do_sleep && APR_BRIGADE_EMPTY(bb)) {
> /* Wait for more (or next call) */
> break;
> }
> _
Much appreciated,
Cory
smime.p7s
Description: S/MIME cryptographic signature
