ngx_http_geo_module range overlaps

Валентин Бартенев vbart at nginx.com
Sat Jan 4 19:03:04 UTC 2014


On Saturday 04 January 2014 22:09:29 Maxim Dounin wrote:
> Hello!
> 
> On Sat, Jan 04, 2014 at 06:12:25PM +0400, Валентин Бартенев wrote:
> > On Saturday 04 January 2014 15:01:16 Ксения Юрьевна Блащук wrote:
> > > Добрый день.
> > > Возник вопрос по модулю ngx_http_geo_module. Почему-то нельзя внести
> > > пересекающиеся диапазоны адресов. В документации:
> > > 
> > > A value of the most specific match is used. For example, for the
> > > 127.0.0.1
> > > address the value "RU" will be chosen, not "US".
> > > 
> > > Example with ranges:
> > > 
> > > geo $country {
> > > 
> > >     ranges;
> > >     default                   ZZ;
> > >     127.0.0.0-127.0.0.0       US;
> > >     127.0.0.1-127.0.0.1       RU;
> > >     127.0.0.1-127.0.0.255     US;
> > >     10.1.0.0-10.1.255.255     RU;
> > >     192.168.1.0-192.168.1.255 UK;
> > > 
> > > }
> > > 
> > > Пытаюсь воспроизвести этот пример:
> > > 
> > > /etc/init.d/nginx reload
> > > 
> > >  * Checking nginx' configuration ...
> > > 
> > > nginx: [emerg] range "127.0.0.1-127.0.0.255" overlaps
> > > "127.0.0.1-127.0.0.1"
> > > in /etc/nginx/sites/test-geo.conf:6
> > > nginx: configuration file /etc/nginx/nginx.conf test failed
> > > nginx: [emerg] range "127.0.0.1-127.0.0.255" overlaps
> > > "127.0.0.1-127.0.0.1"
> > > in /etc/nginx/sites/test-geo.conf:6
> > > nginx: configuration file /etc/nginx/nginx.conf test failed
> > 
> > [..]
> > 
> > > В чем может быть дело?
> > > Спасибо.
> > 
> > Это баг в проверке конфигурации.
> > 
> > Чтобы его обойти можете поменять местами так чтобы вначале шел больший
> > 
> > диапазон:
> >     127.0.0.1-127.0.0.255     US;
> >     127.0.0.1-127.0.0.1       RU;
> 
> Это не баг, это фича.  Код не умеет обрабатывать добавления
> диапазонов, перекрывающих уже существующие диапазоны, и честно об
> этом сообщает.
> 
> При использовании range'ей последующими строками можно
> переопределить часть ранее заданного диапазона адресов.  Задать
> диапазон, который бы включал в себя ранее заданные диапазоны -
> нельзя.

Фича, как известно, это задокументированная бага, а в данном же случае
было не так. =)

Можно и научить:

diff -r b1db6dda3f83 src/http/modules/ngx_http_geo_module.c
--- a/src/http/modules/ngx_http_geo_module.c    Sat Jan 04 18:48:25 2014 +0400
+++ b/src/http/modules/ngx_http_geo_module.c    Sat Jan 04 22:48:11 2014 +0400
@@ -798,10 +798,22 @@ ngx_http_geo_add_range(ngx_conf_t *cf, n
 
             i--;
 
+            if (s < (ngx_uint_t) range[i].start
+                && e == (ngx_uint_t) range[i].end)
+            {
+                e = range[i].start - 1;
+            }
+
             if (e < (ngx_uint_t) range[i].start) {
                 continue;
             }
 
+            if (s == (ngx_uint_t) range[i].start
+                && e > (ngx_uint_t) range[i].end)
+            {
+                s = range[i].end + 1;
+            }
+
             if (s > (ngx_uint_t) range[i].end) {
 
                 /* add after the range */
@@ -835,6 +847,30 @@ ngx_http_geo_add_range(ngx_conf_t *cf, n
                 goto next;
             }
 
+            if (s < (ngx_uint_t) range[i].start
+                && e > (ngx_uint_t) range[i].end)
+            {
+                /* split the new range and insert a fragment after */
+
+                range = ngx_array_push(a);
+                if (range == NULL) {
+                    return NGX_CONF_ERROR;
+                }
+
+                range = a->elts;
+
+                ngx_memmove(&range[i + 2], &range[i + 1],
+                           (a->nelts - 2 - i) * sizeof(ngx_http_geo_range_t));
+
+                range[i + 1].start = range[i].end + 1;
+                range[i + 1].end = (u_short) e;
+                range[i + 1].value = ctx->value;
+
+                e = range[i].start - 1;
+
+                continue;
+            }
+
             if (s > (ngx_uint_t) range[i].start
                 && e < (ngx_uint_t) range[i].end)
             {


Хотя проще видимо задокументировать.

--
Валентин Бартенев


Подробная информация о списке рассылки nginx-ru