Re: nginx-0.1.12

[Igor Sysoev <is@xxxxxxxxxxxxx>]

On Tue, 7 Dec 2004, L0rda wrote:

> Здравствуйте, Igor.
>
> Вы писали 6 декабря 2004 г., 17:44:55:
>
> > Изменения в nginx 0.1.12                                          06.12.2004
>
> >     *) Добавление: параметр лога %request_length.
> bug:
> в конфиге:
>  log_format traffic   '[%time] %request_length %length';
>         location ~* ^.+\.(jpg|jpeg|gif|png|txt|)$
>         {
>             allow all;
>             autoindex    off;
>             expires      30d;
>             root /hosts/www1/html;
>             access_log   /hosts/www1/logs/acct_nginx.log traffic;
>         }
> Запрашиваю картинку, в логе:
> [06/Dec/2004:21:39:50 +0000] 581572656738861517 3145
>
> -----------------------------^^^^^^^^^^^^^^^^^^
> это у всех так, или только у меня?
> или я что-то не так делаю?;)

Это моя ошибка. Нужен такой патчик:

-------------------------
--- src/http/ngx_http_log_handler.c     Mon Dec  6 16:26:39 2004
+++ src/http/ngx_http_log_handler.c     Tue Dec  7 10:20:40 2004
@@ -120,7 +120,7 @@
     { ngx_string("status"), 3, ngx_http_log_status },
     { ngx_string("length"), NGX_OFF_T_LEN, ngx_http_log_length },
     { ngx_string("apache_length"), NGX_OFF_T_LEN, ngx_http_log_apache_length },
-    { ngx_string("request_length"), NGX_OFF_T_LEN,
+    { ngx_string("request_length"), NGX_SIZE_T_LEN,
                                     ngx_http_log_request_length },
     { ngx_string("i"), NGX_HTTP_LOG_ARG, ngx_http_log_header_in },
     { ngx_string("o"), NGX_HTTP_LOG_ARG, ngx_http_log_header_out },
@@ -290,7 +290,7 @@
 static u_char *ngx_http_log_request_length(ngx_http_request_t *r, u_char *buf,
                                            uintptr_t data)
 {
-    return ngx_sprintf(buf, "%O", r->request_length);
+    return ngx_sprintf(buf, "%z", r->request_length);
 }


--- auto/unix   Tue Nov 30 18:56:07 2004
+++ auto/unix   Tue Dec  7 10:24:22 2004
@@ -45,6 +45,7 @@

 ngx_type="size_t"; . auto/types/sizeof
 ngx_param=MAX_SIZE_T_VALUE; ngx_value=$ngx_max_value; . auto/types/value
+ngx_param=NGX_SIZE_T_LEN; ngx_value=$ngx_max_len; . auto/types/value

 ngx_type="off_t"; . auto/types/sizeof
 ngx_param=MAX_OFF_T_VALUE; ngx_value=$ngx_max_value; . auto/types/value
-------------------------


Игорь Сысоев
http://sysoev.ru