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

[Maxim Dounin]

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/