search

CHUNGYUN / flylinkdc

Automatically exported from code.google.com/p/flylinkdc
0 stars 0 forks source link

Несоответствие флага страны и буквенного кода страны в колонке "Расположение" #1315

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
Несоответствие флага страны и буквенного 
кода страны
r502-rc2-x64 build 15520
Когда заполняется три значения (флаг 
страны, название провайдера, буквенный код 
страны) встречается не соответствие флага 
и страны
Примеры

Original issue reported on code.google.com by S.Artyuk...@gmail.com on 26 Sep 2013 at 5:14

Attachments:

GoogleCodeExporter commented 9 years ago 
Просто страна и провайдер опеределяется по 
разным файлам
GeoIPCountryWhois.csv
CustomLocations.ini

Вероятно в них есть несоответствие.
тут я не могу помочь. но вечером точнее 
проверю что те IP корректно вычисляют 
флажки.

Original comment by Pavel.Pimenov@gmail.com on 27 Sep 2013 at 2:25

GoogleCodeExporter commented 9 years ago 
Скриншотик бы где видно несоответствие и IP 
пользователя чтоб было от чего 
отталкиваться.

Original comment by tret2...@gmail.com on 27 Sep 2013 at 3:36

GoogleCodeExporter commented 9 years ago 
Несоответствие флагов и буквенного 
описания стран видно на всех трех 
приложенных выше принтскринах
Вечером после 22 буду дома и выложу скрины с  
колонкой IP.

Original comment by S.Artyuk...@gmail.com on 27 Sep 2013 at 4:43

GoogleCodeExporter commented 9 years ago 
Экраны с IP

Original comment by S.Artyuk...@gmail.com on 27 Sep 2013 at 4:23

Attachments:

GoogleCodeExporter commented 9 years ago 
Думается стоит продублировать ишью тут 
https://www.assembla.com/spaces/customlocations-greylink/tickets/cardwall

Original comment by a.rain...@gmail.com on 27 Sep 2013 at 4:25

GoogleCodeExporter commented 9 years ago 
Зачем? Не в их же базе проблемы, а в 
GeoIPCountryWhois.csv.

Original comment by toss.Alexey on 27 Sep 2013 at 6:01

GoogleCodeExporter commented 9 years ago 
>>Не в их же базе проблемы, а в GeoIPCountryWhois.csv.

Не уверен я в этом. Хотя скорее всего ошибка 
у нас в коде, там перелопатили всё очень 
сильно, при этом внеся в логику работы с 
ГеоИП неоднозначные допущения по поводу 
диапазонов.

Original comment by a.rain...@gmail.com on 27 Sep 2013 at 6:12

GoogleCodeExporter commented 9 years ago 
Может и у вас в коде тоже, но автор 
кастомлока к ним точно отношения не имеет. 
Проверил на вскидку 3 IP с левым флагом с 
первого скриншота по базе максмайнда - 
страна в скобках показана правильно, флаг и 
правда левый.

Original comment by toss.Alexey on 27 Sep 2013 at 6:28

GoogleCodeExporter commented 9 years ago 
Сломал вероятно я. спасибо за баг репорт.

Original comment by Pavel.Pimenov@gmail.com on 27 Sep 2013 at 6:35

GoogleCodeExporter commented 9 years ago 
Паш, глянь в первую очередь именно 
диапазоны, ибо сейчас для кустомлока в 
локальную базу пишется только начало 
диапазона, что является несколько 
опрометчивым допущением в логике, ибо сама 
база не гарантирует, что диапазоны идут 
строго последовательно, и без пропусков.

Original comment by a.rain...@gmail.com on 27 Sep 2013 at 7:32

GoogleCodeExporter commented 9 years ago 
ой, тьфу, не кустомлока, а геоип, кустомлок 
честно парсится как раз.

Original comment by a.rain...@gmail.com on 27 Sep 2013 at 7:32

GoogleCodeExporter commented 9 years ago 
geoIP уже давно по диапазонам хранится 
(появился файл FlylinkDC_locations.sqlite)
до этого 192.168 - показывало как италия.
if (!m_select_geoip.get())
  m_select_geoip = auto_ptr<sqlite3_command>(new sqlite3_command(m_flySQLiteDB,
 "select (select name from fly_dic where id=dic_country), flag_index,start_ip,stop_ip "
 "from location_db.fly_country_ip where start_ip <= ? and stop_ip > ?"));
m_select_geoip.get()->bind(1, __int64(p_ip));

Original comment by Pavel.Pimenov@gmail.com on 27 Sep 2013 at 10:11

GoogleCodeExporter commented 9 years ago 
А, ок :) правда у меня до сих пор локальную 198 
подсеть как Италию показывает Oo. Не пнули 
обновление БД у юзеров что ли?

Original comment by a.rain...@gmail.com on 28 Sep 2013 at 1:32

GoogleCodeExporter commented 9 years ago 
Проверил на дырки GeoIP, в нём 3 дырки
10.*.*.*
127.*.*.*
192.168.*.*
все остальные диапазоны идут без перерывов

Original comment by tret2...@gmail.com on 28 Sep 2013 at 3:14

GoogleCodeExporter commented 9 years ago 
Автору ишуйки скинул адрес обновления 
ночных сборок, одним из последних фиксов в 
той области возможно багу поправили, прошу 
проверить.

Original comment by tret2...@gmail.com on 28 Sep 2013 at 4:31

GoogleCodeExporter commented 9 years ago 
Обновил до r502-rc2-x64 build 15552
С поиском проблем нет как и не было на r15531
Была только на 15528, на более ранних не было
Программа при этом не вылетала, а 
подвисала, не на что не реагировала, 
перейти на другие окна ALT+TAB не давала.  
Завершить можно было только по трем кнопкам

С флагами несоответствие есть до н.в.

Original comment by S.Artyuk...@gmail.com on 28 Sep 2013 at 6:09

Attachments:

GoogleCodeExporter commented 9 years ago 
зашел на тот же хабик, подожду когда список 
IP соберётся большой и посмотрю, 
повторяется ли, если повторится, то с 
накопленным материалом быдет проще найти 
проблему

Original comment by tret2...@gmail.com on 28 Sep 2013 at 5:08

GoogleCodeExporter commented 9 years ago 
У меня это  на всех хабах проявляется

Original comment by S.Artyuk...@gmail.com on 28 Sep 2013 at 5:58

Attachments: