Что не так в этом коде: $headers .= «From: ». $email. «\r\n»;?

Question

MikhailNsk @MikhailNsk

Что не так в этом коде: $headers .= «From: ». $email. «\r\n»;?

Здравствуйте!

На сайте есть форма обратной связи.

Может, я туплю, но ошибки в упор не вижу... Скрипт PHP нормально работал два года, никто ничего не трогал, и вдруг перестал работать.

***

Вот верхний кусок скрипта:

$to = "adress@mail.ru";
$email = $_POST['email'];
$name = $_POST['name'];
$phone = $_POST['phone'];
$msg = $_POST['message'];

$subject = "Запрос с нашего сайта";

$headers = "MIME-Version: 1.0\r\n";

Вопрос задан более трёх лет назад
1089 просмотров

2 комментария

Подписаться 1 Оценить 2 комментария

Решения вопроса 1

1 комментарий

Пригласить эксперта

Ответы на вопрос 1

3 комментария

MikhailNsk @MikhailNsk Автор вопроса

Спасибо за ответ.

И аргументы просматривал, и смотрел выдаёт ли ошибки функция mail(). Ошибок не обнаружил.

Пробовал переставлять код (конкретно "header") местами - не помогает.

Ниже вложил весь код скрипта. Это форма обратной связи с возможностью прикрепить несколько файлов. Может, что подскажете...

(лирическое отступление) В основном я программирую на C# и JS, и там таких проблем у меня нет. А вот когда касаюсь PHP, у меня начинается "боль"... :) Часто решение ищешь тупо методом перебора, не понимая, почему предыдущий способ не работает, а этот вдруг заработал. Понятно, что есть какая-то логика, но чёт для меня она часто неочевидна, например как сейчас. :)

***

<?php

if (isset ($_POST['email']))
{

  $to = "site@mail.ru";
  $email = $_POST['email'];
  $name = $_POST['name'];
  $phone = $_POST['phone'];
  $msg = $_POST['message'];
  
  $subject  = "Запрос с нашего сайта";
  
  $headers  = "MIME-Version: 1.0\r\n";
  $headers .= "From: admin@site.ru\r\n";
  
  $message = "<html><body style='font-family:Tahoma;'>
              <h2 style='font-weight:bold; padding-bottom:10px;'>Запрос с нашего сайта</h2>\r\n
              <p><b>Имя:</b> ".$name."</p>\r\n
              <p><b>E-mail:</b> ".$email."</p>\r\n
              <p><b>Телефон:</b> ".$phone."</p>\r\n
              <p><b>Сообщение:</b> ".$msg."</p>\r\n
              </body></html>";
		 
  $filesize = '';
  $boundary = md5(date('r', time()));
  $headers .= "Content-Type: multipart/mixed; boundary=\"$boundary\"\r\n";
  $message  = "

Content-Type: multipart/mixed; boundary=\"$boundary\"

--$boundary
Content-Type: text/html; charset=\"utf-8\"
Content-Transfer-Encoding: 7bit

$message";
  for($i=0;$i<count($_FILES['file']['name']);$i++) {
      if(is_uploaded_file($_FILES['file']['tmp_name'][$i])) {
         $attachment = chunk_split(base64_encode(file_get_contents($_FILES['file']['tmp_name'][$i])));
         $filename = $_FILES['file']['name'][$i];
         $filetype = $_FILES['file']['type'][$i];
         $filesize += $_FILES['file']['size'][$i];
         $message.="

--$boundary
Content-Type: \"$filetype\"; name=\"$filename\"
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename=\"$filename\"

$attachment";
     }
   }
   $message.="
--$boundary--";

  $allowed = array('image/jpg', 'image/jpeg', 'image/bmp', 'image/gif', 'image/png', 'application/pdf', 'application/msword', 'application/rtf', 'application/xls', 'application/dwg', 'application/pln', 'application/pla');
  
  if ($filesize < 15000000)
  {
    if (in_array($filetype, $allowed) === true || $filesize < 100)
    {
      mail($to, $subject, $message, $headers);
      
      echo "<div class=\"fixed-overlay\" style=\"display:block\" id=\"div1\"><div class=\"modal\" style=\"margin-left:-160px; top:42%\">
	          <div class=\"modal_container green\"><a onclick=\"hidetxt('div1'); return false;\" href=\"#\" title=\"Закрыть\" class=\"close\">X</a>
		        <b style=\"font-size:16px; padding:50px 20px 50px 20px\">Ваш запрос успешно отправлен!</b></div></div></div>";
    }
    
	  else echo "<div class=\"fixed-overlay\" style=\"display:block\" id=\"div2\"><div class=\"modal\" style=\"margin-left:-280px; top:42%\">
	             <div class=\"modal_container red\"><a onclick=\"hidetxt('div2'); return false;\" href=\"#\" title=\"Закрыть\" class=\"close\">X</a>
				       <b>Ошибка, неправильный тип файла(ов).</b> <br><br> (допустимые типы: jpg, jpeg, bmp, gif, png, pdf, msword, rtf, xls, dwg, pln) <br><br> 
				       <b>Попробуйте отправить запрос ещё раз, добавив файлы допустимых типов.</b></div></div></div>";
  }
  
  else echo "<div class=\"fixed-overlay\" style=\"display:block\" id=\"div3\"><div class=\"modal\" style=\"margin-left:-320px; top:42%\">
	           <div class=\"modal_container red\"><a onclick=\"hidetxt('div3'); return false;\" href=\"#\" title=\"Закрыть\" class=\"close\">X</a>
				     <b>Ошибка, размер файла(ов) превышает 15 Мб. <br><br> Попробуйте отправить запрос ещё раз, уменьшив количество файлов или их размер.</b></div></div></div>";
}

?>

Написано более трёх лет назад

lipinart @lipinart

У вас даже элементарной валидации нет. Советую использовать библиотеку PHPMailer

Написано более трёх лет назад

MikhailNsk @MikhailNsk Автор вопроса

Валидация у меня есть. Я использую HTML5

<input type="text" maxlength="40" name="name" autocomplete="on" placeholder="Иван Васильевич" required /><
<input type="email" maxlength="40" name="email" autocomplete="on" placeholder="ivan@mail.ru" required />
<input type="tel" maxlength="20" name="phone" autocomplete="on" placeholder="8-913-987-1234" required />

Написано более трёх лет назад

Ваш ответ на вопрос

Войдите, чтобы написать ответ

Похожие вопросы

Электронная почта

+1 ещё

Средний
Есть способ в мобильных версиях Gmail и Yandex почта отключить темный режим для шаблона письма?
- 1 подписчик
- 7 часов назад
- 20 просмотров
0

ответов
PHP

+1 ещё

Простой
Как обновить PHP на сайте?
- 1 подписчик
- вчера
- 312 просмотров
2

ответа
PHP

+1 ещё

Средний
Yii2-dynamic-form при добавлении строки слетает кодировка. Как решить?
- 1 подписчик
- вчера
- 32 просмотра
1

ответ
PHP

+1 ещё

Простой
Как исправить "Undefined array key «login»,"password" в $_POST?
- 1 подписчик
- вчера
- 96 просмотров
1

ответ
PHP

+3 ещё

Простой
Firefox больше не отправляет в HTTP_ACCEPT image/webp?
- 3 подписчика
- 22 нояб.
- 599 просмотров
2

ответа
PHP

+1 ещё

Простой
Как корректно распределить сумму внутри элементов массива?
- 1 подписчик
- 22 нояб.
- 89 просмотров
1

ответ
PHP

+2 ещё

Сложный
Как обойти блокировку серверов CloudFlare на уровне провайдера?
- 1 подписчик
- 20 нояб.
- 4157 просмотров
6

ответов
PHP

Простой
Как трансформировать массив?
- 1 подписчик
- 20 нояб.
- 136 просмотров
1

ответ
Электронная почта

+1 ещё

Средний
Kyocera M2535dn не может отправить на почту, как решить?
- 1 подписчик
- 20 нояб.
- 56 просмотров
0

ответов
Сетевое администрирование

+4 ещё

Средний
Почему Mail in a box недоступен по внешнему ip?
- 1 подписчик
- 19 нояб.
- 126 просмотров
2

ответа
Показать ещё Загружается…

PHP-разработчик

Decart IT-production

от 260 000 до 340 000 ₽

Разработчик PHP

Автомакон

от 206 000 ₽

Backend-разработчик PHP

Wanted. • Москва

До 160 000 ₽

Изображения на главную страницу в стиле Jetton и 1вин

25 нояб. 2024, в 00:22

12000 руб./за проект

Дизайн презентации

24 нояб. 2024, в 23:46

20000 руб./за проект

Разработать Telegram-бота для анализа звонков продаж

24 нояб. 2024, в 23:15

10000 руб./за проект

Там обычный емэйл адрес из

print_r($email) выводит электронный адрес

В том то и дело, что всё вроде в порядке, никаких ошибок скрипт не выдаёт. Как написал ранее, скрипт два года работал вообще без проблем, никто ничего в нём не менял, и вдруг перестал...

При этом, если добавить символ ">" в

$headers .= "From: " . $email . ">\r\n";

то всё снова работает (и ещё пара примеров в первом сообщении)

Возможно, это связано с обработкой заголовков почтовыми сервиса (письма не приходят ни на Яндекс, ни на Mail.ru). А может, я не вижу какой-то очевидной мелочи... Но похоже никто пока не увидел.

Answer 1 · 2017-08-04 13:19:08

Нельзя во From использовать чужие e-mail адреса, это нарушает DMARC и для доменов со строгой DMARC-политикой (mail.ru, yahoo.com) письма доходить не будут. Ставьте в From и конверте адреса, которые ваш сервер авторизован отправлять, настройте для низ SPF и DKIM. Если вам нужно отвечать на письма, то ставьте адрес пользователя в Reply-To.

Answer 2 · 2017-08-03 07:50:32

lipinart @lipinart

Порядок тоже важен. Перед функцией mail() распечатай все ее аргументы и увидишь ошибку.

Ответ написан более трёх лет назад

3 комментария

Что не так в этом коде: $headers .= «From: ». $email. «\r\n»;?

Войдите, чтобы написать ответ

Минуточку внимания

Войдите на сайт