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

Projects
None yet
1 participant
@pavel-pimenov
Owner

pavel-pimenov commented Aug 25, 2015

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

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

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

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

Status: Accepted

@pavel-pimenov

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

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

@pavel-pimenov

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

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

@pavel-pimenov

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

Экраны с IP

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

@pavel-pimenov

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

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

@pavel-pimenov

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

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

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

@pavel-pimenov

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

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

@pavel-pimenov

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

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

@pavel-pimenov

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

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

@pavel-pimenov

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

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

@pavel-pimenov

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

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

@pavel-pimenov

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

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

@pavel-pimenov

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

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

@pavel-pimenov

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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

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

@pavel-pimenov

This comment has been minimized.

Owner

pavel-pimenov commented Aug 25, 2015

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