On Tue, Sep 19, 2017 at 12:13 PM, Yann Ylavic <ylavic....@gmail.com> wrote:
>
> On Tue, Sep 19, 2017 at 11:02 AM, Luca Toscano <toscano.l...@gmail.com> wrote:
>
>> Index: server/util_expr_eval.c
>> ===================================================================
>> --- server/util_expr_eval.c (revision 1808842)
>> +++ server/util_expr_eval.c (working copy)
>> @@ -1044,7 +1044,12 @@
>> t = ctx->r->headers_in;
>> else { /* req, http */
>> t = ctx->r->headers_in;
>> - add_vary(ctx, arg);
>> + /* Skip the 'Vary: Host' header combination
>> + * as indicated in rfc7231 section-7.1.4
>> + */
>> + if (strcmp(arg, "Host")){
>> + add_vary(ctx, arg);
>> + }
>> }
>> return apr_table_get(t, arg);
>> }
>
> This one looks good.
Well, headers' names are case insensitive, so strcasecmp() above ;)
>
>
> Regards,
> Yann.
