Re: nginx-0.3.31

[Igor Sysoev <is@xxxxxxxxxxxxx>]

On Sat, 11 Mar 2006, Igor Sysoev wrote:

> On Sat, 11 Mar 2006, Alexey Polyakov wrote:
>
> > Что-то в этой версии разломалось касающееся работы с server_name.
> > 0.3.29 при запуске nginx -t выдает как и ожидалось:
> > 2006/03/10 16:19:59 [info] 19554#0: the configuration file
> > /usr/local/nginx/conf/nginx.conf syntax is ok
> > 2006/03/10 16:19:59 [info] 19554#0: the configuration file
> > /usr/local/nginx/conf/nginx.conf was tested successfully
> > 0.3.31 выдает для почти всех имен сервера сначала:
> > 2006/03/10 16:20:54 [alert] 19705#0: server name "имя"
> > (подряд несколько тысяч записей)
> > далее для всех тех же записей:
> > 2006/03/10 16:20:54 [alert] 19705#0: wildcard server name "имя"
> > и уже после всего этого
> > 2006/03/10 16:20:54 [info] 19705#0: the configuration file
> > /usr/local/nginx/conf/nginx.conf syntax is ok
> > 2006/03/10 16:20:54 [info] 19705#0: the configuration file
> > /usr/local/nginx/conf/nginx.conf was tested successfully
> >
> > Вилдкардов я не использую. Причем ругается не на все имена хостов - в
> > файле, куда я вывод nginx -t перенаправил - 35014 строк образовалось,
> > т. е. на 17507 имен ругнулся, а в файле, где я держу имена серверов -
> > 19652 имени.
> Это я забыл убрать отладочный лог из релиза.
> Прилагаемый патч лечит это дело.

Патч я тоже забыл добавить :)


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

--- src/http/ngx_http.c Thu Mar  9 14:22:15 2006
+++ src/http/ngx_http.c Sat Mar 11 09:24:31 2006
@@ -580,9 +580,6 @@
             name = in_addr[a].names.elts;
             for (s = 0; s < in_addr[a].names.nelts; s++) {
 
-                ngx_log_error(NGX_LOG_ALERT, cf->log, 0,
-                              "server name \"%V\"", &name[s].name);
-
                 ch = name[s].name.data[0];
 
                 if (ch == '*' || ch == '.') {
@@ -604,9 +601,6 @@
             }
 
             for (s = 0; s < in_addr[a].names.nelts; s++) {
-
-                ngx_log_error(NGX_LOG_ALERT, cf->log, 0,
-                              "wildcard server name \"%V\"", &name[s].name);
 
                 ch = name[s].name.data[0];