Что не так в этом коде: $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";

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

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 />

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

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

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

Войти через центр авторизации

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

PHP

Средний
Php повторные Калбеки?
- 1 подписчик
- 7 минут назад
- 5 просмотров
0

ответов
PHP

+1 ещё

Простой
Как в php формировать ответ на AJAX XMLHttpRequest запрос?
- 1 подписчик
- 3 часа назад
- 51 просмотр
0

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

+2 ещё

Простой
Можно ли узнать список почтовых ящиков в моем собственном домене?
- 2 подписчика
- 3 часа назад
- 55 просмотров
2

ответа
PHP

+2 ещё

Средний
Запросы soap в инфоклинику на php?
- 1 подписчик
- 6 часов назад
- 103 просмотра
2

ответа
PHP

Простой
Не работает часть скрипта PHP при смены PHP 7 на 8?
- 1 подписчик
- 20 часов назад
- 171 просмотр
3

ответа
PHP

Средний
Как найти в массиве ответа API нужное значение, при том что значение может быть написано в разном регистре?
- 1 подписчик
- вчера
- 133 просмотра
3

ответа
PHP

+1 ещё

Простой
Как отправить сообщение в определенную тему в группе Telegram боту на PHP?
- 1 подписчик
- вчера
- 68 просмотров
0

ответов
PHP

+1 ещё

Простой
Почему не работает JWT поверка?
- 1 подписчик
- вчера
- 67 просмотров
0

ответов
PHP

+1 ещё

Средний
Как отладить плавающий баг проверки капчи?
- 1 подписчик
- вчера
- 58 просмотров
1

ответ
PHP

Простой
Как вывести переменную php которая выводит путь к файлу в строке $data = File('assets/files/uslugi-i-ceny/558.csv');?
- 1 подписчик
- вчера
- 99 просмотров
0

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

PHP Developer

YCLIENTS • Москва

от 200 000 до 350 000 ₽

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

Ведисофт • Екатеринбург

от 25 000 ₽

Midlle PHP developer (backend)

ИТЦ Аусферр • Магнитогорск

от 100 000 до 160 000 ₽

Доработка парсера

23 апр. 2024, в 19:21

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

Верстка сайта

23 апр. 2024, в 19:05

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

Создать учебный сайт на React

23 апр. 2024, в 19:01

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

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

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»;?

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

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

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