simple email generator - Githubissues

HochuOlivie commented 4 years ago

Методы обычно пишутся с маленькой буквы, а классы с большой.

Enhisir commented 4 years ago

Методы обычно пишутся с маленькой буквы, а классы с большой.

fixed (https://github.com/osint-dev-team/osint-framework/pull/9/commits/5cb74140668dbff898103f30af8a929e2516cdc3)

Enhisir commented 4 years ago

Сама обработка юзернейма крутая, и мутации прикольные, исходя из того, что я вижу в результатах!

Есть некоторые фундаментальные проблемы:

Во-первых, нам не нужно 673 домена. Нам достаточно взять топ-10 или топ-50, для начала: google, yandex, mail, yahoo, microsoft, etc., без каких-нибудь ChessFan.net и других специфичных

Максимальное число e-mail-ов не должно превышать 1.000, в идеале 500 адресов, чтобы мы могли это потом безболезненно скормить email_verifier модулю и узнать, какие из них существуют (и при этом сделать это относительно быстро, но это потом)

Реверс имени типа "john.doe.adams" -> "adams.doe.john" это круто, но надо оценить, насколько это применимо в реальной жизни? Как я вижу, такие перестановки могут очень сильно увеличить конечный пул результатов, поэтому, может их пока лучше "отключить" и посмотреть на количество с обычными мутациями типа -, _, ., etc.?

Модуль нужно привести к новому виду, как остальные скрипты

Модуль нужно перенести в папку convert/, т.к. он будет запускаться немного иначе и будет своего рода прослойкой для генерации результатов

Код из __init__.py нужно убрать в module.py в рамках приведения кода к новому виду

После этого можно вернуться к самому алгоритму обработки. А так, в общем, круто! Буду ждать изменений.

fixed(513b800f57ee87c8e48264bb25ee74e3bbec0680)

Enhisir commented 4 years ago

Попробуй запустить как модуль - не работает. Пути ломаные.
(venv) ➜  convert git:(email_generator) ✗ pwd
/osint-framework/src/scripts/convert
(venv) ➜  convert git:(email_generator) ✗ python3 -m email_generator test.this.name
{'message': "[Errno 2] No such file or directory: 'settings/settings.json'",
 'result': None,
 'status': 'error'}
Остальные замечания отметил. Буду ждать исправлений!

fixed(88c8025a37707e71017c0a407a66830818976495)

Enhisir commented 4 years ago

Требуются небольшие фиксы.

fixed(28c75db)

osint-dev-team / osint-framework

simple email generator #9