[PHP]walidacja załącznika, wysyła mimo warunku Opcje

[jaroslavmo]

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 
 
$maxsize = 2*1024*1024
 
if(isset($_Files['file']['name'])) {
 
    if($_FILES['file']['size'] >  $maxsize) {
        echo 'zbyt duży plik';
        exit;
 
    }  else {
        $mail->AddAttachment($_FILES['file']['tmp_name'], $_FILES['file']['name']);
    }
 
}
 
 
 
[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 
 
 
 
<?php 
ini_set('display_errors', '1');
ini_set('display_startup_errors', '1');
error_reporting(E_ALL);
 
$honeypot = $_POST['honey'];
$nameU = $_POST['name'];
$email = $_POST['email'];
$content = $_POST['message'];
$phoneU = $_POST['phone'];
$fileUpload = $_POST['file'];
 
use PHPMailer\PHPMailer\PHPMailer;
use PHPMailer\PHPMailer\SMTP;
use PHPMailer\PHPMailer\Exception;
 
require 'autoload.php';
 
if ($_SERVER["REQUEST_METHOD"] == "POST") {
    // robot detection
    $honeypot = trim($_POST["honey"]);     
 
    if(!empty($honeypot)) {
      echo "NO SPAM!"; 
      exit;
    }
 
$mail = new PHPMailer; //From email address and name 
$mail->From = $email ; 
 
$mail->FromName = $nameU; //To address and name 
$mail->addAddress("jaroslaw.mor@gmail.com", "Vomo");//Recipient name is optional
 
$mail->isHTML(true); 
$mail->Subject = "Zapytanie ze strony www"; 
$mail->Body = "Telefon:$phoneU<br><br>Treść wiadomośći:<br>$content";
$mail->AltBody = "Telefon:$phoneU\n$content"; 
 
$maxsize = 2*1024*1024;
 
 
 
if(isset($_Files['file']['name'])) {
 
    if($_FILES['file']['size'] >  $maxsize) {
        echo 'zbyt duży plik';
        exit;
 
    }  else {
        $mail->AddAttachment($_FILES['file']['tmp_name'], $_FILES['file']['name']);
    }
 
}
 
 
 
if(!$mail->send()) 
{
    echo "Mailer Error: " . $mail->ErrorInfo;
} 
else 
{
    header("Location: sent.html");
    exit();
}
 
}
 
 
 
 
 
 
 
[PHP] pobierz, plaintext 

[nospor]

ZNajdz 10 roznic

$_Files
$_FILES

[jaroslavmo]

Dzięki za wskazówkę, poprawiłem ale niestety nadal tak samo.

[nospor]

To sprawdz co dokladnie dostajesz w $_FILES

print_r($_FILES);
exit;

[jaroslavmo]

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]

No i chyba juz widzisz ze twoje file to uploded-file a nie file

$_FILES['uploded-file']

[jaroslavmo]

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 
 
for ($i=0; $i < count($attachments) ; $i++) { 
    $fileTmpName = $_FILES['uploaded-file']["tmp_name"][$i]; 
    $filename = $_FILES['uploaded-file']["name"][$i];
    move_uploaded_file($fileTmpName,  "uploads/" . $filename);
     $mail-> AddAttachment("uploads/". $filename); 
}
 
 
[PHP] pobierz, plaintext 

Natomiast ten fragment

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

wywołuje ostrzeżenie

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 
 
 
<?php 
ini_set('display_errors', '1');
ini_set('display_startup_errors', '1');
error_reporting(E_ALL);
 
$honeypot = $_POST['honey'];
$nameU = $_POST['name'];
$email = $_POST['email'];
$content = $_POST['message'];
$phoneU = $_POST['phone'];
//$fileUpload = $_POST['file'];
 
use PHPMailer\PHPMailer\PHPMailer;
use PHPMailer\PHPMailer\SMTP;
use PHPMailer\PHPMailer\Exception;
 
require 'autoload.php';
 
if ($_SERVER["REQUEST_METHOD"] == "POST") {
    // robot detection
    $honeypot = trim($_POST["honey"]);     
 
    if(!empty($honeypot)) {
      echo "NO SPAM!"; 
      exit;
    }
 
$mail = new PHPMailer; //From email address and name 
$mail->From = $email ; 
 
$mail->FromName = $nameU; //To address and name 
$mail->addAddress("jaroslaw.mor@gmail.com");//Recipient name is optional
 
$mail->isHTML(true); 
$mail->Subject = "Zapytanie ze strony www"; 
$mail->Body = "Telefon:$phoneU<br><br>Treść wiadomośći:<br>$content";
$mail->AltBody = "Telefon:$phoneU\n$content"; 
$maxsize = 2*1024*1024;
$attachments =  $_FILES['uploaded-file']["name"];
$acceptedFormats = array("doc", " docx" , "pdf" , "txt" , "odt",  "zip", "7z", "rar", "jpg", "jpeg");
$uploadedFile = $_FILES['uploaded-file']['tmp_name'];
 
 
 
for ($i=0; $i < count($attachments) ; $i++) { 
    $fileTmpName = $_FILES['uploaded-file']["tmp_name"][$i]; 
    $filename = $_FILES['uploaded-file']["name"][$i];
    move_uploaded_file($fileTmpName,  "uploads/" . $filename);
    $mail-> AddAttachment("uploads/". $filename);
 
 
}
 
 
 
if (isset($attachments)) {
 
    if(filesize($filename) > $maxsize) {
        $message ="plik jest zbyt duży";
    }
 
 
    else if (!in_array(pathinfo($fileTmpName, PATHINFO_EXTENSION),  $acceptedFormats)) {
        $message ="nieprawidłowy format pliku";
 
} else {
    $message = '';
 
}
}
 
 
 
if(!$mail->send()) 
{
    echo "Mailer Error: " . $mail->ErrorInfo;
    die();
} 
else 
{
    //header("Location: sent.html");
 
    print_r($_FILES);
 
        die();
} 
 
 
 
 
}
 
[PHP] pobierz, plaintext 

Ten post edytował jaroslavmo 21.03.2022, 22:00:30

[nospor]

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]

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 
 
  if(isset($_FILES['uploaded-file'])) {
 
        for ($i = 0; $i < count($_FILES['uploaded-file']['name']); $i++) {
 
            if ($_FILES['uploaded-file']['error'] !== UPLOAD_ERR_OK) continue;
            $fileTmpName = $_FILES['uploaded-file']["tmp_name"][$i]; 
            $filename = $_FILES['uploaded-file']["name"][$i];
 
 
 
            if($_FILES['uploaded-file']["name"]['size'][$i] > $maxsize) {
                exit('plik jest zbyt duży');
            } else {
 
            move_uploaded_file($fileTmpName,  "uploads/" . $filename);
            $mail-> AddAttachment("uploads/". $filename);
 
            }
 
 
 
 
    }//loop end
 
    }//isset end
 
 
 
 
 
 
}//post end
 
[PHP] pobierz, plaintext 

całość

[PHP] pobierz, plaintext 
 
<?php 
ini_set('display_errors', '1');
ini_set('display_startup_errors', '1');
error_reporting(E_ALL);
 
$honeypot = $_POST['honey'];
$nameU = $_POST['name'];
$email = $_POST['email'];
$content = $_POST['message'];
$phoneU = $_POST['phone'];
//$fileUpload = $_POST['file'];
$honeypot = trim($_POST["honey"]); 
 
use PHPMailer\PHPMailer\PHPMailer;
use PHPMailer\PHPMailer\SMTP;
use PHPMailer\PHPMailer\Exception;
 
require 'autoload.php';
 
if ($_SERVER["REQUEST_METHOD"] == "POST") {
 
	$mail = new PHPMailer; //From email address and name 
    $mail->From = $email ; 
 
    $mail->FromName = $nameU; //To address and name 
    $mail->addAddress("jaroslaw.mor@gmail.com");//Recipient name is optional
 
    $mail->isHTML(true); 
    $mail->Subject = "Zapytanie ze strony www"; 
    $mail->Body = "Telefon:$phoneU<br>Treść wiadomośći:<br><br>$content";
    $mail->AltBody = "Telefon:$phoneU\n$content"; 
    $maxsize = 2*1024*1024;
    $attachments =  $_FILES['uploaded-file']["name"];
    $acceptedFormats = array("doc", " docx" , "pdf" , "txt" , "odt",  "zip", "7z", "rar", "jpg", "jpeg");
 
  if(!empty($honeypot)) {
      echo "NO SPAM!"; 
      exit;
    } 
 
    if(isset($_FILES['uploaded-file'])) {
 
        for ($i = 0; $i < count($_FILES['uploaded-file']['name']); $i++) {
 
            if ($_FILES['uploaded-file']['error'] !== UPLOAD_ERR_OK) continue;
            $fileTmpName = $_FILES['uploaded-file']["tmp_name"][$i]; 
            $filename = $_FILES['uploaded-file']["name"][$i];
 
 
 
            if($_FILES['uploaded-file']["name"]['size'][$i] > $maxsize) {
                exit('plik jest zbyt duży');
            } else {
 
            move_uploaded_file($fileTmpName,  "uploads/" . $filename);
            $mail-> AddAttachment("uploads/". $filename);
 
            }
 
 
 
 
    }//loop end
 
    }//isset end
 
 
 
 
 
 
}//post end
 
 
 
 
 
if(!$mail->send()) {
      echo "Mailer Error: " . $mail->ErrorInfo;
     exit();
} 
 
		else {
        header("Location: sent.html");
 
           exit();
} 
 
 
[PHP] pobierz, plaintext 

[nospor]

$_FILES['uploaded-file']["name"]['size'][$i]

[jaroslavmo]

$_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.

Ten post edytował jaroslavmo 22.03.2022, 23:24:45

[nospor]

$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]

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 
 
 
 
<?php
//Import PHPMailer classes into the global namespace
//These must be at the top of your script, not inside a function
use PHPMailer\PHPMailer\PHPMailer;
use PHPMailer\PHPMailer\SMTP;
use PHPMailer\PHPMailer\Exception;
 
//Load Composer's autoloader
require 'autoload.php';
 
//Create an instance; passing `true` enables exceptions
$mail = new PHPMailer(true);
$honeypot = $_POST['honey'];
$user_name = $_POST['name'];
$user_email = $_POST['email'];
$user_message = $_POST['message'];
$user_phone = $_POST['phone'];
$honeypot = trim($_POST["honey"]);
$max_size = 2 * 1024 * 1204; //2mb
$attachment = $_FILES['uploaded-file'];
 
if ($_SERVER["REQUEST_METHOD"] == "POST") {
 
 
    if(!empty($honeypot)) {
        echo "NO SPAM!"; 
        exit;
      }    else {
 
        $mail = new PHPMailer; //From email address and name 
        $mail->isMail(); 
 
        //sender
        $mail->From = $user_email;
        $mail->FromName = $user_name;
 
        //recipient
        $mail->addAddress("jaroslaw.mor@gmail.com");
 
 
        //mail subject
        $mail->Subject = "Zapytanie ze strony www"; 
 
 
        $mail->isHTML(true);
        //body mail 
        $mail->Body = "Telefon:$user_phone<br><br>Treść wiadomośći:<br>$user_message";
        $mail->AltBody = "Telefon:$user_phone\n$content"; 
 
        //attachment
        if(isset($attachment)) {
 
            for ($i = 0; $i < count($_FILES['uploaded-file']['name']); $i++) {
                if ($_FILES['uploaded-file']['error'][$i] !== UPLOAD_ERR_OK) continue;
                $file_TmpName = $_FILES['uploaded-file']["tmp_name"][$i]; 
                $file_name = $_FILES['uploaded-file']["name"][$i];  
 
                if ($_FILES['uploaded-file']['size'][$i] > $maxsize) { 
 
                    $message = urlencode("załącznik przekorczył limit 10MB");
                    header("Location:".$_SERVER['HTTP_REFERER']. '#file-size-error');
                    die();
 
                             }
 
                else{
                 move_uploaded_file($fileTmpName,  "uploads/" . $filename);
                 $mail-> AddAttachment("uploads/". $filename);
                                                                }      
            }//for
        }//isset
 
        if(!$mail->send()) {
            echo "Mailer Error: " . $mail->ErrorInfo;
           exit();
      } 
 
              else {
              header("Location: sent.html");
 
                 exit();
 
      }//if send else
 
}//honey else end
 
}//post end
 
 
 
 
 
 
 
[PHP] pobierz, plaintext 

[nospor]

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 (IMG:style_emoticons/default/wink.gif)

[jaroslavmo]

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 (IMG:style_emoticons/default/wink.gif)

dzięki link pomógł we wykryciu kolejnego babola, teraz wszystko działa (IMG:style_emoticons/default/smile.gif)