pavel-pimenov / flylinkdc-r5xx

flylinkdc-r5xx
GNU General Public License v2.0
55 stars 27 forks source link

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

Open pavel-pimenov opened 9 years ago

pavel-pimenov commented 9 years ago

From S.Artyuk...@gmail.com on September 26, 2013 19:14:27

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

Attachment: флаги_01.jpg флаги_02.jpg флаги_03.jpg

Original issue: http://code.google.com/p/flylinkdc/issues/detail?id=1315

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on September 26, 2013 19:25:40

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

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

Status: Accepted

pavel-pimenov commented 9 years ago

From tret2...@gmail.com on September 26, 2013 20:36:19

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

pavel-pimenov commented 9 years ago

From S.Artyuk...@gmail.com on September 26, 2013 21:43:28

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

pavel-pimenov commented 9 years ago

From S.Artyuk...@gmail.com on September 27, 2013 09:23:31

Экраны с IP

Attachment: флаги_04.jpg флаги_05.jpg флаги_06.jpg

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on September 27, 2013 09:25:59

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

Summary: Несоответствие флага страны и буквенного кода страны в колонке "Расположение" (was: Несоответствие флага страны и буквенного кода страны)

pavel-pimenov commented 9 years ago

From toss.Alexey on September 27, 2013 11:01:44

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

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on September 27, 2013 11:12:11

Не в их же базе проблемы, а в GeoIPCountryWhois.csv.

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

pavel-pimenov commented 9 years ago

From toss.Alexey on September 27, 2013 11:28:20

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

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on September 27, 2013 11:35:04

Сломал вероятно я. спасибо за баг репорт.

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on September 27, 2013 12:32:14

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

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on September 27, 2013 12:32:42

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

pavel-pimenov commented 9 years ago

From Pavel.Pimenov@gmail.com on September 27, 2013 15:11:00

geoIP уже давно по диапазонам хранится (появился файл FlylinkDC_locations.sqlite) до этого 192.168 - показывало как италия. if (!m_select_geoip.get()) m_select_geoip = auto_ptr(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));

pavel-pimenov commented 9 years ago

From a.rain...@gmail.com on September 27, 2013 18:32:29

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

pavel-pimenov commented 9 years ago

From tret2...@gmail.com on September 27, 2013 20:14:11

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

pavel-pimenov commented 9 years ago

From tret2...@gmail.com on September 27, 2013 21:31:02

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

pavel-pimenov commented 9 years ago

From S.Artyuk...@gmail.com on September 27, 2013 23:09:25

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

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

Attachment: флаги_07.jpg

pavel-pimenov commented 9 years ago

From tret2...@gmail.com on September 28, 2013 10:08:43

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

pavel-pimenov commented 9 years ago

From S.Artyuk...@gmail.com on September 28, 2013 10:58:50

У меня это на всех хабах проявляется

Attachment: флаги_08.jpg флаги_09.jpg флаги_10.jpg