osint-dev-team / osint-framework

:fork_and_knife: All-in-one OSINT-RECON Swiss Knife
https://mca.nsu.ru/projects/#rec196282976
GNU General Public License v2.0
34 stars 4 forks source link

Added verification by phone number whether a person is registered in VK. #8

Closed SN4KEBYTE closed 4 years ago

SN4KEBYTE commented 4 years ago

Кешна, суперкринж:

  1. Сделано не по инструкции. Тебе нужен класс Runner, который наследуется от OsintRunner, и в методе run писать свой код
  2. Чекер через селениум -- некруто. А в особенности некруто, когда пользователю нужно указывать путь до своего браузера

PS я не куратор, но такой код точно не примут

  1. Дельное замечание, спасибо. Исправлю :)
  2. Через VK API такое не провернуть, Selenium тут неплохо себя показывает (без пути к хромдрайверу не обойтись). А как можно сделать лучше?

P. S. оценивать, кринж или некринж, точно не тебе.

HochuOlivie commented 4 years ago

Тебе сейчас нужно просто смириться, что придется переписать код, потому что его точно не примут. Есть отличная библиотека requests, которая позволяет делать HTTP-запросы. Почитай про то, что такое HTTP-запрос, как его делать с помощью Python и сделай. Это займет строчек 5-10. (Я точно знаю, что там нет ничего сложного/нет трудных непонятных параметров, потому что с помощью запросов авторизовался в VK пару лет назад)

P.S. Я, конечно, очень извиняюсь, но кринж -- объективная оценка, к сожалению. Ты решил просто забить на кроссплатформенность. Разве ты видел хоть одну тулзу, которая бы просила у тебя ввести путь до своего браузера? Я -- нет. Я не запускал твой код, но, вроде, ты даже запускаешь хром не в headless режиме. Уже не говорю о блокировании запросов с img/css, которые все очень сильно замедляют. Selenium используется в основном для автоматических тестов, и в проекте такого плана ему нет места быть (по моему, конечно же, субъективному мнению).

SN4KEBYTE commented 4 years ago

1) Я знаю, что такое HTTP запрос и знаю, что существует такая замечательная библиотека как requests. 2) Антон показывал мне, как это сделать запросами, но там есть довольно заморочный момент, который обойти сразу не получилось. Я подумаю, как это можно сделать. 3) Я до этого не интересовался ИБ и никаками тулзами не пользовался. С Selenium работаю впервые и про headless - режим и блокирование картинок и css не осведомлен, почитаю. Одним словом, это абсолютно незнакомая для меня область.

Благодарю за критику и дельные замечания, но твои "объективные" оценки меня мало интересуют. Извини, если задел чем-то.

manmolecular commented 4 years ago

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

manmolecular commented 4 years ago

Плюс, сам модуль нужно переименовать во что-то типа check_vk_reg, покороче

manmolecular commented 4 years ago

@SN4KEBYTE привет! Как успехи? Нужна помощь? 🔔

SN4KEBYTE commented 4 years ago

@manmolecular привет :) Я немного увлекся своим проектом, завтра вернусь к работе.

manmolecular commented 4 years ago

@manmolecular привет :) Я немного увлекся своим проектом, завтра вернусь к работе.

Окей, отлично! 👍

manmolecular commented 4 years ago
Снимок экрана 2020-08-07 в 00 31 16
SN4KEBYTE commented 4 years ago

@manmolecular получается, что путь к драйверу неправильно вычисляется?

manmolecular commented 4 years ago

@manmolecular получается, что путь к драйверу неправильно вычисляется?

Да, всё так. Видимо, вместо бинаря он пытается запустить папку, в которой бинарь лежит. Но всё равно с правами разобраться надо.