[PATCH] SPDY: fix support for headers with multiple values

Piotr Sikora piotr at cloudflare.com
Tue Jul 8 10:12:51 UTC 2014


Hey Valentin,

> Probably, it's better to use constant here (like r->state = 0 in
> the same function):
>
> diff -r 9d3a9c45fc43 src/http/ngx_http_spdy.c
> --- a/src/http/ngx_http_spdy.c  Tue Jul 01 20:52:08 2014 +0400
> +++ b/src/http/ngx_http_spdy.c  Tue Jul 08 13:55:52 2014 +0400
> @@ -2584,6 +2584,8 @@ ngx_http_spdy_parse_header(ngx_http_requ
>                  r->header_end = p;
>                  r->header_in->pos = p + 1;
>
> +                r->state = sw_value;
> +
>                  return NGX_OK;
>              }
>
>

That's fine with me.

Best regards,
Piotr Sikora



More information about the nginx-devel mailing list