Nginx-ru mailing list archive (nginx-ru@sysoev.ru)
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: ngx_http_geo_module range overlaps
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 mailing list
nginx-ru@xxxxxxxxx
http://mailman.nginx.org/mailman/listinfo/nginx-ru
|