Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP]walidacja załącznika
Forum PHP.pl > Forum > Przedszkole
jaroslavmo
21.03.2022, 17:36:05
Użyłem php mailer do wysyłania emaila, chciałbym zrobić walidację wielkości pliku po stronie serwera.

stowrzyłem taki warunek

[PHP] pobierz, plaintext 
  1.  
  2. $maxsize = 2*1024*1024
  3.  
  4. if(isset($_Files['file']['name'])) {
  5.  
  6.     if($_FILES['file']['size'] >  $maxsize) {
  7.         echo 'zbyt duży plik';
  8.         exit;
  9.  
  10.     }  else {
  11.         $mail->AddAttachment($_FILES['file']['tmp_name'], $_FILES['file']['name']);
  12.     }
  13.  
  14. }
  15.  
  16.  
  17.  
[PHP] pobierz, plaintext


mimo tego mail się wysyła bez załącznika, ale wysyła. Chciałbym to zmienić.
Poniżej cały mój kod php





[PHP] pobierz, plaintext 
  1.  
  2.  
  3.  
  4. <?php 
  5. ini_set('display_errors', '1');
  6. ini_set('display_startup_errors', '1');
  7. error_reporting(E_ALL);
  8.  
  9. $honeypot = $_POST['honey'];
  10. $nameU = $_POST['name'];
  11. $email = $_POST['email'];
  12. $content = $_POST['message'];
  13. $phoneU = $_POST['phone'];
  14. $fileUpload = $_POST['file'];
  15.  
  16. use PHPMailer\PHPMailer\PHPMailer;
  17. use PHPMailer\PHPMailer\SMTP;
  18. use PHPMailer\PHPMailer\Exception;
  19.  
  20. require 'autoload.php';
  21.  
  22. if ($_SERVER["REQUEST_METHOD"] == "POST") {
  23.     // robot detection
  24.     $honeypot = trim($_POST["honey"]);     
  25.  
  26.     if(!empty($honeypot)) {
  27.       echo "NO SPAM!"; 
  28.       exit;
  29.     }
  30.  
  31. $mail = new PHPMailer; //From email address and name 
  32. $mail->From = $email ; 
  33.  
  34. $mail->FromName = $nameU; //To address and name 
  35. $mail->addAddress("jaroslaw.mor@gmail.com", "Vomo");//Recipient name is optional
  36.  
  37. $mail->isHTML(true); 
  38. $mail->Subject = "Zapytanie ze strony www"; 
  39. $mail->Body = "Telefon:$phoneU<br><br>Treść wiadomośći:<br>$content";
  40. $mail->AltBody = "Telefon:$phoneU\n$content"; 
  41.  
  42. $maxsize = 2*1024*1024;
  43.  
  44.  
  45.  
  46. if(isset($_Files['file']['name'])) {
  47.  
  48.     if($_FILES['file']['size'] >  $maxsize) {
  49.         echo 'zbyt duży plik';
  50.         exit;
  51.  
  52.     }  else {
  53.         $mail->AddAttachment($_FILES['file']['tmp_name'], $_FILES['file']['name']);
  54.     }
  55.  
  56. }
  57.  
  58.  
  59.  
  60. if(!$mail->send()) 
  61. {
  62.     echo "Mailer Error: " . $mail->ErrorInfo;
  63. } 
  64. else 
  65. {
  66.     header("Location: sent.html");
  67.     exit();
  68. }
  69.  
  70. }
  71.  
  72.  
  73.  
  74.  
  75.  
  76.  
  77.  
[PHP] pobierz, plaintext

nospor
21.03.2022, 17:49:37
ZNajdz 10 roznic

$_Files
$_FILES
jaroslavmo
21.03.2022, 18:06:45
Dzięki za wskazówkę, poprawiłem ale niestety nadal tak samo.
nospor
21.03.2022, 18:07:28
To sprawdz co dokladnie dostajesz w $_FILES

print_r($_FILES);
exit;
jaroslavmo
21.03.2022, 18:14:47
Cytat(nospor @ 21.03.2022, 18:07:28 ) *
To sprawdz co dokladnie dostajesz w $_FILES

print_r($_FILES);
exit;




Array ( [uploded-file] => Array ( [name] => Tropico 5 V1.10 Trainer +6.rar [type] => application/octet-stream [tmp_name] => /tmp/www/mork/phpqbDtSk [error] => 0 [size] => 4585995 ) ) 1
nospor
21.03.2022, 18:15:24
No i chyba juz widzisz ze twoje file to uploded-file a nie file

$_FILES['uploded-file']
jaroslavmo
21.03.2022, 21:58:41
By nie tworzyć nowego wątku, zapomniałem że miał być więcej niż jeden załącznik. Uwzględniając poprzednie wskazówki pętlę do załączników.

Stworzyłem pętle załączniki widzi i nie zgłasza problemu, jak kodu od walidacji nie ma, wysyła wiadomość ze załącznikami. 

[PHP] pobierz, plaintext 
  1.  
  2. for ($i=0; $i < count($attachments) ; $i++) { 
  3.     $fileTmpName = $_FILES['uploaded-file']["tmp_name"][$i]; 
  4.     $filename = $_FILES['uploaded-file']["name"][$i];
  5.     move_uploaded_file($fileTmpName,  "uploads/" . $filename);
  6.      $mail-> AddAttachment("uploads/". $filename); 
  7. }
  8.  
  9.  
[PHP] pobierz, plaintext




Natomiast ten fragment 

[PHP] pobierz, plaintext 
  1.  
  2. if(filesize($filename) > $maxsize) {
  3.     $message ="plik jest zbyt duży";
  4. }
  5.  
[PHP] pobierz, plaintext


wywołuje ostrzeżenie

Cytat
Warning: filesize(): stat failed for oświadczenie-o-przekazaniu-praw-majatkowych.pdf in /home/mork/public_html/vomo/mailer.php on line 55 Array ( [uploaded-file] => Array ( [name] => Array ( [0] => Klauzule informacyjne i okna zgody - wzór.pdf [1] => oświadczenie-o-przekazaniu-praw-majatkowych.pdf ) [type] => Array ( [0] => application/pdf [1] => application/pdf ) [tmp_name] => Array ( [0] => /tmp/www/mork/phpSW91R4 [1] => /tmp/www/mork/phpOAykxZ ) [error] => Array ( [0] => 0 [1] => 0 ) [size] => Array ( [0] => 148245 [1] => 56805 ) ) )



całość kodu

[PHP] pobierz, plaintext 
  1.  
  2.  
  3. <?php 
  4. ini_set('display_errors', '1');
  5. ini_set('display_startup_errors', '1');
  6. error_reporting(E_ALL);
  7.  
  8. $honeypot = $_POST['honey'];
  9. $nameU = $_POST['name'];
  10. $email = $_POST['email'];
  11. $content = $_POST['message'];
  12. $phoneU = $_POST['phone'];
  13. //$fileUpload = $_POST['file'];
  14.  
  15. use PHPMailer\PHPMailer\PHPMailer;
  16. use PHPMailer\PHPMailer\SMTP;
  17. use PHPMailer\PHPMailer\Exception;
  18.  
  19. require 'autoload.php';
  20.  
  21. if ($_SERVER["REQUEST_METHOD"] == "POST") {
  22.     // robot detection
  23.     $honeypot = trim($_POST["honey"]);     
  24.  
  25.     if(!empty($honeypot)) {
  26.       echo "NO SPAM!"; 
  27.       exit;
  28.     }
  29.  
  30. $mail = new PHPMailer; //From email address and name 
  31. $mail->From = $email ; 
  32.  
  33. $mail->FromName = $nameU; //To address and name 
  34. $mail->addAddress("jaroslaw.mor@gmail.com");//Recipient name is optional
  35.  
  36. $mail->isHTML(true); 
  37. $mail->Subject = "Zapytanie ze strony www"; 
  38. $mail->Body = "Telefon:$phoneU<br><br>Treść wiadomośći:<br>$content";
  39. $mail->AltBody = "Telefon:$phoneU\n$content"; 
  40. $maxsize = 2*1024*1024;
  41. $attachments =  $_FILES['uploaded-file']["name"];
  42. $acceptedFormats = array("doc", " docx" , "pdf" , "txt" , "odt",  "zip", "7z", "rar", "jpg", "jpeg");
  43. $uploadedFile = $_FILES['uploaded-file']['tmp_name'];
  44.  
  45.  
  46.  
  47. for ($i=0; $i < count($attachments) ; $i++) { 
  48.     $fileTmpName = $_FILES['uploaded-file']["tmp_name"][$i]; 
  49.     $filename = $_FILES['uploaded-file']["name"][$i];
  50.     move_uploaded_file($fileTmpName,  "uploads/" . $filename);
  51.     $mail-> AddAttachment("uploads/". $filename);
  52.  
  53.  
  54. }
  55.  
  56.  
  57.  
  58. if (isset($attachments)) {
  59.  
  60.     if(filesize($filename) > $maxsize) {
  61.         $message ="plik jest zbyt duży";
  62.     }
  63.  
  64.  
  65.     else if (!in_array(pathinfo($fileTmpName, PATHINFO_EXTENSION),  $acceptedFormats)) {
  66.         $message ="nieprawidłowy format pliku";
  67.  
  68. } else {
  69.     $message = '';
  70.  
  71. }
  72. }
  73.  
  74.  
  75.  
  76. if(!$mail->send()) 
  77. {
  78.     echo "Mailer Error: " . $mail->ErrorInfo;
  79.     die();
  80. } 
  81. else 
  82. {
  83.     //header("Location: sent.html");
  84.  
  85.     print_r($_FILES);
  86.  
  87.         die();
  88. } 
  89.  
  90.  
  91.  
  92.  
  93. }
  94.  
[PHP] pobierz, plaintext
nospor
22.03.2022, 08:33:10
1) Walidacja ma byc w petli a nie poza nia. No to chyba dosc logiczne. Poza petla to ty co najwyzej ostatni element mozesz zwalidowac

2)
if(filesize($filename) > $maxsize) {
A co to ma byc? przeciez $filename to nazwa pliku a nie sciezka do pliku. Komunikat bledy wyraznie ci o tym mowi. Poza tym, rozmiar przeciez masz nadal w $_FILES. Czemu z tego nie korzystasz? Czemu nafle przy petli stwierdziles ze bedziesz uzywal czegos innego? Co sie zminilo? No nic sie niezmienilo
jaroslavmo
22.03.2022, 21:36:12
zmieniłem to trochę, błędami nie sypie ale nawet jeśli waga któregoś ze załączników przekracza 2mb to i tak maila wysyła tyle że bez załączników

kod odpowiedzialny za załącznik:

[PHP] pobierz, plaintext 
  1.  
  2.   if(isset($_FILES['uploaded-file'])) {
  3.  
  4.         for ($i = 0; $i < count($_FILES['uploaded-file']['name']); $i++) {
  5.  
  6.             if ($_FILES['uploaded-file']['error'] !== UPLOAD_ERR_OK) continue;
  7.             $fileTmpName = $_FILES['uploaded-file']["tmp_name"][$i]; 
  8.             $filename = $_FILES['uploaded-file']["name"][$i];
  9.  
  10.  
  11.  
  12.             if($_FILES['uploaded-file']["name"]['size'][$i] > $maxsize) {
  13.                 exit('plik jest zbyt duży');
  14.             } else {
  15.  
  16.             move_uploaded_file($fileTmpName,  "uploads/" . $filename);
  17.             $mail-> AddAttachment("uploads/". $filename);
  18.  
  19.             }
  20.  
  21.  
  22.  
  23.  
  24.     }//loop end
  25.  
  26.     }//isset end
  27.  
  28.  
  29.  
  30.  
  31.  
  32.  
  33. }//post end
  34.  
[PHP] pobierz, plaintext




całość 

[PHP] pobierz, plaintext 
  1.  
  2. <?php 
  3. ini_set('display_errors', '1');
  4. ini_set('display_startup_errors', '1');
  5. error_reporting(E_ALL);
  6.  
  7. $honeypot = $_POST['honey'];
  8. $nameU = $_POST['name'];
  9. $email = $_POST['email'];
  10. $content = $_POST['message'];
  11. $phoneU = $_POST['phone'];
  12. //$fileUpload = $_POST['file'];
  13. $honeypot = trim($_POST["honey"]); 
  14.  
  15. use PHPMailer\PHPMailer\PHPMailer;
  16. use PHPMailer\PHPMailer\SMTP;
  17. use PHPMailer\PHPMailer\Exception;
  18.  
  19. require 'autoload.php';
  20.  
  21. if ($_SERVER["REQUEST_METHOD"] == "POST") {
  22.  
  23. 	$mail = new PHPMailer; //From email address and name 
  24.     $mail->From = $email ; 
  25.  
  26.     $mail->FromName = $nameU; //To address and name 
  27.     $mail->addAddress("jaroslaw.mor@gmail.com");//Recipient name is optional
  28.  
  29.     $mail->isHTML(true); 
  30.     $mail->Subject = "Zapytanie ze strony www"; 
  31.     $mail->Body = "Telefon:$phoneU<br>Treść wiadomośći:<br><br>$content";
  32.     $mail->AltBody = "Telefon:$phoneU\n$content"; 
  33.     $maxsize = 2*1024*1024;
  34.     $attachments =  $_FILES['uploaded-file']["name"];
  35.     $acceptedFormats = array("doc", " docx" , "pdf" , "txt" , "odt",  "zip", "7z", "rar", "jpg", "jpeg");
  36.  
  37.   if(!empty($honeypot)) {
  38.       echo "NO SPAM!"; 
  39.       exit;
  40.     } 
  41.  
  42.     if(isset($_FILES['uploaded-file'])) {
  43.  
  44.         for ($i = 0; $i < count($_FILES['uploaded-file']['name']); $i++) {
  45.  
  46.             if ($_FILES['uploaded-file']['error'] !== UPLOAD_ERR_OK) continue;
  47.             $fileTmpName = $_FILES['uploaded-file']["tmp_name"][$i]; 
  48.             $filename = $_FILES['uploaded-file']["name"][$i];
  49.  
  50.  
  51.  
  52.             if($_FILES['uploaded-file']["name"]['size'][$i] > $maxsize) {
  53.                 exit('plik jest zbyt duży');
  54.             } else {
  55.  
  56.             move_uploaded_file($fileTmpName,  "uploads/" . $filename);
  57.             $mail-> AddAttachment("uploads/". $filename);
  58.  
  59.             }
  60.  
  61.  
  62.  
  63.  
  64.     }//loop end
  65.  
  66.     }//isset end
  67.  
  68.  
  69.  
  70.  
  71.  
  72.  
  73. }//post end
  74.  
  75.  
  76.  
  77.  
  78.  
  79. if(!$mail->send()) {
  80.       echo "Mailer Error: " . $mail->ErrorInfo;
  81.      exit();
  82. } 
  83.  
  84. 		else {
  85.         header("Location: sent.html");
  86.  
  87.            exit();
  88. } 
  89.  
  90.  
[PHP] pobierz, plaintext
nospor
22.03.2022, 22:40:40
$_FILES['uploaded-file']["name"]['size'][$i]
jaroslavmo
22.03.2022, 23:20:21
Cytat(nospor @ 22.03.2022, 22:40:40 ) *
$_FILES['uploaded-file']["name"]['size'][$i]


$FILES w moim warunku wygląda dokładanie tak jak to napisałeś.

nie wiem o co chodzi.

Php to nie moja działka, tak wyszło że musiałem się sam jakoś kombinować, metodą prób i błędów, googla, youtuba i stackoverflow coś wyskrobałem. Jak to k....stwo zacznie działać nikomu nie będę zawracać d.......y o php , bo się więcej tego nie tykam.
nospor
23.03.2022, 07:30:22
Cytat
$FILES w moim warunku wygląda dokładanie tak jak to napisałeś.

Ja to dokladnie wiem. Ja ci podalem ten kawalek kodu gdzie wlasnie masz blad i masz poprawic.
jaroslavmo
27.03.2022, 11:33:15
Ok, poradziłem sobie z załącznikiem i działało a teraz mam taki dziwny problem że każdy załącznik, nawet jeśli jest mniejszy od maxsize to i tak pokazuje że plik jest zbyt duży i wykonuje się instrukcja dla zbyt dużego pliku. 





[PHP] pobierz, plaintext 
  1.  
  2.  
  3.  
  4. <?php
  5. //Import PHPMailer classes into the global namespace
  6. //These must be at the top of your script, not inside a function
  7. use PHPMailer\PHPMailer\PHPMailer;
  8. use PHPMailer\PHPMailer\SMTP;
  9. use PHPMailer\PHPMailer\Exception;
  10.  
  11. //Load Composer's autoloader
  12. require 'autoload.php';
  13.  
  14. //Create an instance; passing `true` enables exceptions
  15. $mail = new PHPMailer(true);
  16. $honeypot = $_POST['honey'];
  17. $user_name = $_POST['name'];
  18. $user_email = $_POST['email'];
  19. $user_message = $_POST['message'];
  20. $user_phone = $_POST['phone'];
  21. $honeypot = trim($_POST["honey"]);
  22. $max_size = 2 * 1024 * 1204; //2mb
  23. $attachment = $_FILES['uploaded-file'];
  24.  
  25. if ($_SERVER["REQUEST_METHOD"] == "POST") {
  26.  
  27.  
  28.     if(!empty($honeypot)) {
  29.         echo "NO SPAM!"; 
  30.         exit;
  31.       }    else {
  32.  
  33.         $mail = new PHPMailer; //From email address and name 
  34.         $mail->isMail(); 
  35.  
  36.         //sender
  37.         $mail->From = $user_email;
  38.         $mail->FromName = $user_name;
  39.  
  40.         //recipient
  41.         $mail->addAddress("jaroslaw.mor@gmail.com");
  42.  
  43.  
  44.         //mail subject
  45.         $mail->Subject = "Zapytanie ze strony www"; 
  46.  
  47.  
  48.         $mail->isHTML(true);
  49.         //body mail 
  50.         $mail->Body = "Telefon:$user_phone<br><br>Treść wiadomośći:<br>$user_message";
  51.         $mail->AltBody = "Telefon:$user_phone\n$content"; 
  52.  
  53.         //attachment
  54.         if(isset($attachment)) {
  55.  
  56.             for ($i = 0; $i < count($_FILES['uploaded-file']['name']); $i++) {
  57.                 if ($_FILES['uploaded-file']['error'][$i] !== UPLOAD_ERR_OK) continue;
  58.                 $file_TmpName = $_FILES['uploaded-file']["tmp_name"][$i]; 
  59.                 $file_name = $_FILES['uploaded-file']["name"][$i];  
  60.  
  61.                 if ($_FILES['uploaded-file']['size'][$i] > $maxsize) { 
  62.  
  63.                     $message = urlencode("załącznik przekorczył limit 10MB");
  64.                     header("Location:".$_SERVER['HTTP_REFERER']. '#file-size-error');
  65.                     die();
  66.  
  67.                              }
  68.  
  69.                 else{
  70.                  move_uploaded_file($fileTmpName,  "uploads/" . $filename);
  71.                  $mail-> AddAttachment("uploads/". $filename);
  72.                                                                 }      
  73.             }//for
  74.         }//isset
  75.  
  76.         if(!$mail->send()) {
  77.             echo "Mailer Error: " . $mail->ErrorInfo;
  78.            exit();
  79.       } 
  80.  
  81.               else {
  82.               header("Location: sent.html");
  83.  
  84.                  exit();
  85.  
  86.       }//if send else
  87.  
  88. }//honey else end
  89.  
  90. }//post end
  91.  
  92.  
  93.  
  94.  
  95.  
  96.  
  97.  
[PHP] pobierz, plaintext
nospor
27.03.2022, 13:58:24
Znajdz 10 roznic

$maxsize
$max_size

Zapoznaj sie prosze i zastosuj do tego tematu
Temat: Jak poprawnie zadac pytanie oraz przygotowac srodowisko pracy

Jak zobacze jeszcze jeden post z powodu nie istniejacych zmiennych, to bede zamykal wink.gif
jaroslavmo
27.03.2022, 21:19:21
Cytat(nospor @ 27.03.2022, 14:58:24 ) *
Znajdz 10 roznic

$maxsize
$max_size

Zapoznaj sie prosze i zastosuj do tego tematu
Temat: Jak poprawnie zadac pytanie oraz przygotowac srodowisko pracy

Jak zobacze jeszcze jeden post z powodu nie istniejacych zmiennych, to bede zamykal wink.gif


dzięki link pomógł we wykryciu kolejnego babola, teraz wszystko działa smile.gif
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.