Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
pavel-pimenov opened this issue Aug 25, 2015 · 18 comments

Comments

@pavel-pimenov
Copy link
Owner

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
Copy link
Owner Author

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

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

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

Status: Accepted

@pavel-pimenov
Copy link
Owner Author

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

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

@pavel-pimenov
Copy link
Owner Author

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

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

@pavel-pimenov
Copy link
Owner Author

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

Экраны с IP

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

@pavel-pimenov
Copy link
Owner Author

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
Copy link
Owner Author

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

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

@pavel-pimenov
Copy link
Owner Author

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

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

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

@pavel-pimenov
Copy link
Owner Author

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

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

@pavel-pimenov
Copy link
Owner Author

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

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

@pavel-pimenov
Copy link
Owner Author

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

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

@pavel-pimenov
Copy link
Owner Author

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

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

@pavel-pimenov
Copy link
Owner Author

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<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));

@pavel-pimenov
Copy link
Owner Author

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

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

@pavel-pimenov
Copy link
Owner Author

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

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

@pavel-pimenov
Copy link
Owner Author

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

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

@pavel-pimenov
Copy link
Owner Author

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
Copy link
Owner Author

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

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

@pavel-pimenov
Copy link
Owner Author

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

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant