search

ValdikSS / blockcheck

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

Add isup.me check #41

Closed Acharvak closed 8 years ago

Acharvak commented 8 years ago

To prevent the user from being confused in the middle of the process if Python is too old and to prevent false results if some sites are down

Acharvak commented 8 years ago

Сразу оговорюсь, что тестировалось в трёх ситуациях:

  1. На провайдере, который ничего не блокирует
  2. На провайдере, который блокирует по IP
  3. Без подключения к интернету

Во время проверки оба сайта с HTTPS как раз лежали.

Скрипт не падает, результаты ожидаемые. Но в других ситуациях на практике не тестировалось. Хотя вроде всё правильно.

ValdikSS commented 8 years ago

Нужна ли проверка версии Python? Код не работает на Python ниже 3.3? Если работает, пожалуйста, уберите проверку. И сделайте, пожалуйста, rebase в один коммит.

Acharvak commented 8 years ago

rebase потом сделаю.

Конкретно добавленные строчки скорее всего будут работать на 3.3, но ведь _get_url с сертификатом не будет?

ValdikSS commented 8 years ago

https://github.com/ValdikSS/blockcheck/commit/cdfb65af9c5b5d383b82c13fa9005555d3fd210c

ValdikSS commented 8 years ago

Если есть в системе /etc/ssl/certs/ca-certificates.crt, то проверка сертификатов заработает и на Python 3.2.

Acharvak commented 8 years ago

Понятно. Тогда уберу.

ValdikSS commented 8 years ago

Не закрывайте pull request, он не копирует ваш репозиторий. Все изменения, которые вы делаете в своем branch своего форка после открытия pull request сразу же отражаются в pull request.

Acharvak commented 8 years ago

Даже если я сделаю rebase и push --force?

Хорошо.

ValdikSS commented 8 years ago

Да, именно так и сделайте.

Acharvak commented 8 years ago

Запилил, перебазировал на текущую HEAD — вроде работает.

ValdikSS commented 8 years ago

Посмотрите, пожалуйста, branch isup.me.

Acharvak commented 8 years ago

Почему вы всё время используете \t? У меня в терминале, например, таб = 8 пробелов, поэтому лог получается плохо читаемый. Если вы рассчитываете на определённый размер таба, то, я думаю, лучше вместо таба использовать пробелы. Или надо в строках типа "[⁇] Ошибка при соединении" вместо пробела после [⁇] ставить \t.

Кроме того, у Рутрекера нет в коде страницы "static.rutracker.", видимо, они теперь поменяли на "static.t-ru.org". Это нужно исправить, а то скрипт считает, что Рутрекер всё время лежит.

Ещё написал ответы на строчные комментарии выше.

Я не совсем понимаю, что вы решили насчёт логики в test_http_access? Вы написали, что она вам не нравится, но никак её не изменили. Edit: а, нет, вижу, что изменили.

ValdikSS commented 8 years ago

У меня тоже таб 8 пробелов, и если раньше мне нравилось такое форматирование, то сейчас уже не очень. Давайте в этом pull request оставим форматирование как есть, а потом подумаем, как его лучше поправить.

Acharvak commented 8 years ago

isup.me почему-то перестал работать с HTTPS-сайтами. Пример:

http://isup.me/http://google.com/ - работает http://isup.me/https://google.com/ - чушь собачья

Мне кажется, раньше оно работало. Тем более, что про неизвестные ему протоколы (например, FTP) он так и говорит: «doesn't look like a site on the interwho».

Что делать?

ValdikSS commented 8 years ago

Смержено в 1b3d86e4687ced4783c60010bf3abaff9ce62ffe, спасибо!