search

ValdikSS / blockcheck

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

detect and ignore teredo/6to4 #57

Closed nshopik closed 7 years ago

nshopik commented 7 years ago

Ой это не то

nshopik commented 7 years ago

Надо поспать уже, тороплюсь, переделаю завтра

ValdikSS commented 7 years ago

Еще нужно сделать параметр --force-ipv6 для принудительного включения. Эту проверку лучше добавить в main.

nshopik commented 7 years ago

Проверил на Window10 с teredo все корректно определяеться и игнорируеться и форсируеться если задать аргумент

ValdikSS commented 7 years ago

Вам приходят уведомления о моих комментариях к коду?

nshopik commented 7 years ago

Вам приходят уведомления о моих комментариях к коду?

Да все вижу

nshopik commented 7 years ago

ipaddress.IPv6Address(v6src).teredo is None and ipaddress.IPv6Address(v6src).sixtofour is None: Нужно чтобы было True в варианте без проверки оно будет False, поменял стиль в соотвествии с рекомендации PEP8

ValdikSS commented 7 years ago

if not ipaddress.IPv6Address(v6src).teredo and not ipaddress.IPv6Address(v6src).sixtofour. Пожалуйста, добавьте еще слово «туннель» в текст, чтобы человек, который не знает, что такое Teredo или 6to4, хотя бы отдаленно догадался: : обнаружен туннель Teredo или 6to4, игнорируем.

nshopik commented 7 years ago

Вроде все. Не внимательно смотрел на комментарий про строчку not ipaddress.IPv6Address(v6src).teredo and not ipaddress.IPv6Address(v6src).sixtofour

ValdikSS commented 7 years ago

У вас говно влезло.

nshopik commented 7 years ago

Вижу уже, сейчас уберу

nshopik commented 7 years ago

Достаточно будет нового коммита без левых файлов, если merge сделаете они не попадут из истории в репу вашу? Или мне надо делать rebase уже?

ValdikSS commented 7 years ago

Достаточно, уберите их следующим коммитом просто.

ValdikSS commented 7 years ago

Класс, спасибо!

nshopik commented 7 years ago

Вам спасибо, провели по коду за руку и научили даже чему-то.

Я не настоящий сварщик, поэтому только быдлокод получаеться, лишь бы работало