Re: Fwd: патч для включения доп. заголовков на коды ошибок
Илья Шипицин
chipitsine на gmail.com
Сб Май 21 18:42:06 MSD 2011
какая, например, проблема возникает с данным патчем ? можно на примере ?
21 мая 2011 г. 14:03 пользователь Maxim Dounin <mdounin на mdounin.ru> написал:
> Hello!
>
> On Sat, May 21, 2011 at 12:24:11PM +0600, Илья Шипицин wrote:
>
> > Добрый день!
> >
> > еще раз предлагаю патч для заголовков.
>
> Проблема по прежнему та же самая, но в этот раз для людей,
> пытающихся выставлять заголовки через add_header.
>
> IMHO, в этом месте единственное безопасное решение - отдать
> контроль пользователю.
>
> Maxim Dounin
>
> >
> > ---------- Пересланное сообщение ----------
> > От кого: Илья Шипицин <chipitsine на gmail.com>
> > Дата: 13 мая 2011 г. 0:02
> > Тема: Re: патч для включения доп. заголовков на коды ошибок
> > Кому: nginx-ru на nginx.org
> >
> >
> > извиняюсь за задержку ответа, руки не доходили. при изложенных
> > обстоятельствах правка предлагается следующая "отдавать заголовки на коды
> > ошибок, если это не заголовки expire":
> >
> > --- src/http/modules/ngx_http_headers_filter_module.c.orig 2011-05-12
> > 23:58:10.867870746 +0600
> > +++ src/http/modules/ngx_http_headers_filter_module.c 2011-05-12
> > 23:58:29.131868824 +0600
> > @@ -141,18 +141,18 @@
> >
> > conf = ngx_http_get_module_loc_conf(r,
> ngx_http_headers_filter_module);
> >
> > - if ((conf->expires == NGX_HTTP_EXPIRES_OFF && conf->headers == NULL)
> > - || r != r->main
> > - || (r->headers_out.status != NGX_HTTP_OK
> > - && r->headers_out.status != NGX_HTTP_NO_CONTENT
> > - && r->headers_out.status != NGX_HTTP_MOVED_PERMANENTLY
> > - && r->headers_out.status != NGX_HTTP_MOVED_TEMPORARILY
> > - && r->headers_out.status != NGX_HTTP_NOT_MODIFIED))
> > + if ((conf->expires == NGX_HTTP_EXPIRES_OFF && conf->headers == NULL)
> ||
> > r != r->main)
> > {
> > return ngx_http_next_header_filter(r);
> > }
> >
> > - if (conf->expires != NGX_HTTP_EXPIRES_OFF) {
> > + if (conf->expires != NGX_HTTP_EXPIRES_OFF &&
> > + (r->headers_out.status == NGX_HTTP_OK ||
> > + r->headers_out.status == NGX_HTTP_NO_CONTENT ||
> > + r->headers_out.status == NGX_HTTP_MOVED_PERMANENTLY ||
> > + r->headers_out.status == NGX_HTTP_MOVED_TEMPORARILY ||
> > + r->headers_out.status == NGX_HTTP_NOT_MODIFIED))
> > + {
> > if (ngx_http_set_expires(r, conf) != NGX_OK) {
> > return NGX_ERROR;
> > }
> >
> >
> >
> > 19 апреля 2011 г. 10:39 пользователь Igor Sysoev <igor на sysoev.ru>
> написал:
> >
> > On Tue, Apr 19, 2011 at 09:03:32AM +0600, Илья Шипицин wrote:
> > > > expires max;
> > > >
> > > > по логике отвечает за контент, размещенный непосредственно на nginx,
> > > > например, редко меняющуюся статику. статика размещенная на nginx на
> может
> > > > давать 500-е ошибки.
> > >
> > > Может. Например, когда заканчиваются файлы, сокеты и тому подобное.
> > >
> > >
> > > --
> > > Igor Sysoev
> > >
> > > _______________________________________________
> > > nginx-ru mailing list
> > > nginx-ru на nginx.org
> > > http://nginx.org/mailman/listinfo/nginx-ru
> > >
>
> > _______________________________________________
> > nginx-ru mailing list
> > nginx-ru на nginx.org
> > http://nginx.org/mailman/listinfo/nginx-ru
>
>
> _______________________________________________
> nginx-ru mailing list
> nginx-ru на nginx.org
> http://nginx.org/mailman/listinfo/nginx-ru
>
----------- следущая часть -----------
Вложение в формате HTML было извлечено…
URL: <http://nginx.org/pipermail/nginx-ru/attachments/20110521/5bb489bf/attachment.html>
Подробная информация о списке рассылки nginx-ru