Welcome! Log In Create A New Profile

Advanced

Re: Теперь нельзя выставлять тип контента!

Maxim Dounin
August 08, 2014 10:04AM
Hello!

On Thu, Aug 07, 2014 at 11:49:01AM -0400, rand257 wrote:

> Доброго времени суток!
>
> Извиняюсь за поднятие столь старой темы, но она полностью соответствует
> моему вопросу.
>
> Итак:
> Имеется nginx, php-fpm и сайт файлохранилища, в котором для выдачи
> статических файлов используется заголовок X-Accel-Redirect в отдельный
> специальный локейшен. В этом локейшене выставлен post_action на вызов
> php-скрипта, подсчитывающий статистику по трафику с выполненного запроса и
> количеству завершенных/незавершенных скачиваний файлов. После обновления
> nginx с некоей доисторической версии на современную 1.6.1 скрипт статистики
> более не вызывается, в логах имеется упоминавшийся здесь алерт "header
> already sent while reading response".

Это выглядит как ошибка, post_action не должен приводить к
срабатыванию этой проверки, т.к. заголовок на самом деле второй
раз не посылает.

Патч:

--- a/src/http/ngx_http_core_module.c
+++ b/src/http/ngx_http_core_module.c
@@ -1972,7 +1972,7 @@ ngx_http_send_response(ngx_http_request_
ngx_int_t
ngx_http_send_header(ngx_http_request_t *r)
{
- if (r->header_sent) {
+ if (r->header_sent && !r->post_action) {
ngx_log_error(NGX_LOG_ALERT, r->connection->log, 0,
"header already sent");
return NGX_ERROR;

--
Maxim Dounin
http://nginx.org/

_______________________________________________
nginx-ru mailing list
nginx-ru@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-ru
Subject Author Posted

Теперь нельзя выставлять тип контента!

nickolay September 21, 2013 01:40PM

Re: Теперь нельзя выставлять тип контента!

Maxim Dounin September 21, 2013 07:14PM

Re: Теперь нельзя выставлять тип контента!

nickolay September 21, 2013 07:36PM

Re: Теперь нельзя выставлять тип контента!

Maxim Dounin September 22, 2013 02:22PM

Re: Теперь нельзя выставлять тип контента!

rand257 August 07, 2014 11:49AM

Re: Теперь нельзя выставлять тип контента!

Илья Шипицин August 07, 2014 01:26PM

Re: Теперь нельзя выставлять тип контента!

Maxim Dounin August 08, 2014 10:04AM



Sorry, only registered users may post in this forum.

Click here to login

Online Users

Guests: 77
Record Number of Users: 6 on February 13, 2018
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready