search

ValdikSS / blockcheck

Russian ISP blocking type checker NOTE: NOT WORKING CURRENTLY. ВНИМАНИЕ: НЕ РАБОТАЕТ НА ТЕКУЩИЙ МОМЕНТ
MIT License
1.37k stars 144 forks source link

address family mask hiding improvment #54

Closed nshopik closed 7 years ago

nshopik commented 7 years ago

Используя модуль ipaddress, скрываем адреса по маске /24 и /64 соответственно

ValdikSS commented 7 years ago

Сделайте тогда по маске /60 для IPv6, чтобы была точность до провайдера, а не до клиента.

nshopik commented 7 years ago

С точностью до провайдера это тогда лучше наверно /48 меньше анонснов в bgp не бывает глобальном

ValdikSS commented 7 years ago

Добавьте еще xxx в конец адреса, как сейчас сделано, чтобы было понятно, что адрес маскируется.

nshopik commented 7 years ago

Маска предполагает что адрес не полный. Иначе тогда опять конвертируем в str и делаем тоже самое что было до патча. Мне лично xxxx часть не нравиться как выглядит, будто глюк определения чтоли.

Возможно имеет смысл добавить надпись ваш адрес был суммаризован до маски [/24,/48] и здесь не показан

ValdikSS commented 7 years ago

Возможно имеет смысл добавить надпись ваш адрес был суммаризован до маски [/24,/48] и здесь не показан

Лучше не стоит. Я стараюсь писать софт для обычных людей, которые не знают, что такое маска подсети, поэтому считаю неуместным вывод настолько подробной поясняющей информации, напрямую не относящейся к тестам.

И, честно сказать, мой вариант с xxx мне нравится больше. Но не отрицаю, что код маскировки IPv6 требует доработки.

nshopik commented 7 years ago

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

Моя цель была сделать кусок кода чуть лучше читаемым/меньше и убрать проблемную часть маскировки IPv6 конечно. И модуль в будущем нужен еще будет для другой части

nshopik commented 7 years ago

Обсуждая #55 я подумал если мы будем определять ASN, зачем нам тогда IP адрес показывать. Имеет смысл показать названия оператора и его ASN, все равно тип блокировок всегда в пределах ASN свой

ValdikSS commented 7 years ago

зачем нам тогда IP адрес показывать.

Чтобы точнее узнать регион, т.к. такая информация во whois не всегда есть. Обращаться к 2ip.ru в любом случае нужно, у них актуальные базы провайдеров. Когда во whois ерунда, у 2ip.ru правильное название провайдера.

nshopik commented 7 years ago

Я имел ввиду конечному пользователю можно и не показывать его и не отправлять его. Запрос мы и так будем делать конечно, но показывать сохранять только ASN + название\регион оператора то что отдает 2ip.

Если хотите оставить можно попробовать указывать его настоящую маску а не статичные /24 и /48 всегда (CIDR брать из RDAP). Плюс добавим текст Ваш полный адрес был скрыт. Как такой вариант патча?

ValdikSS commented 7 years ago

Я не понимаю, зачем это нужно. Обычному пользователю показывать ASN ни к чему, мне он тоже не нужен: ASN из IP получить можно, а обратно — нельзя. У одной AS может быть много разных диапазонов IP-адресов. Замена IP на ASN не позволит мне определить регион в результатах тестирования (2ip чаще показывает только название провайдера, без региона).

Показывать маску тоже смысла не вижу.

nshopik commented 7 years ago

Make sense, я совсем забыл про мегачемпионов, где ASN натянут на всю страну.