Иногда админу сайта приходили письма с содержанием:
-"This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a permanent error. The following address(es) failed:" и т. д.
Полагаю, что это роботы пытались "пригласить друга".
Поставил защиту (по своему разумению) в файл modules\mod_invite\module.php добавил строки
$yur_email = cmsCore::request('yur_email', 'str', '');
if ($yur_email) $errors = true;
а в файл templates\_default_\modules\mod_invite.tpl добавил строку
<input type="hidden" name="yur_email" value="" />
Так после этого писем стало намного больше.
Это случайное совпадение или с кодом я накосячил?
InstantCMS 1.X
#1
12 апреля 2017 в 22:57
#2
13 апреля 2017 в 08:44
Это и т.д. самое важное из этого. Ни какого отношения к icms данное сообщение, скорее всего, не имеет. Настраивайте почту на сервере или обратитесь к хостеру. Это довольно расспостранённая ошибка почтового сервера.The following address(es) failed:" и т. д.
Собственно, тут два вопроса.
Нет ли ошибок в коде, который я вставил?
И второй вопрос. Увеличение количества таких писем может быть как-то связано с изменением кода?
Было раньше 1 — 2 в месяц, после изменения кода 1 — 2 в день, а сегодня за ночь аж 15 штук.
В коде письма видны участники сообщения, в том числе есть и скрипт движка.
Вот "дословный" текст письма.
Нет ли ошибок в коде, который я вставил?
И второй вопрос. Увеличение количества таких писем может быть как-то связано с изменением кода?
Было раньше 1 — 2 в месяц, после изменения кода 1 — 2 в день, а сегодня за ночь аж 15 штук.
В коде письма видны участники сообщения, в том числе есть и скрипт движка.
Вот "дословный" текст письма.
Сегодня в 20:51
#4
13 апреля 2017 в 15:29
В коде вы не указали value. Выходит переменная приходит пустой и error== false. Так же дожен бть такой знак ==, а не =. Ну а вообще данный код ничего не решает. У вас не настроена почта на сервере. И у user25 к вам почта не приходит, вот от сюда и сообщения.
#5
13 апреля 2017 в 16:05
Так и задумано, если человек, то переменная придёт пустой, а если робот, то есть надежда что он заполнит и это скрытое поле, а значит в php-коде, по аналогии с родным кодом, например,
if (!$email){
cmsCore::addSessionMessage($_LANG['ERR_NEED_MAIL'], 'error'); $errors = true;
}
я и написал if ($yur_email) $errors = true;
if (!$email){
cmsCore::addSessionMessage($_LANG['ERR_NEED_MAIL'], 'error'); $errors = true;
}
я и написал if ($yur_email) $errors = true;
#6
13 апреля 2017 в 16:19
Насчёт $errors = true;, да всё верно, это я тормознул. Со всем остальным смотрите сами. Нравится изобретать велосипед, удачи.
P.S. С полями я лично ничего не понял. Если оно скрытое, как его кто то заполнит или не заполнит?
P.S. С полями я лично ничего не понял. Если оно скрытое, как его кто то заполнит или не заполнит?
Если скрытое, то ни как (без специальных способов) человек не заполнит, да и не надо заполнять, а робот не смотрит (полагаю) на страницу, а читает код, создаёт у себя в памяти аналогичную форму, заполняет все поля, и отправляет эту свою копию формы, не нажимая кнопки "Отправить" на странице.Если оно скрытое, как его кто то заполнит или не заполнит?
#8
13 апреля 2017 в 18:54
Понятно. Но тогда наверное лучше прятать не js а через css. Да и боты говорят не ведутся сейчас на hidden