Can anyone see anything wrong with this code?

[tdob]

The code below is an upload script, which essentially uploads a file and checks the file type and file size.

For the most part this works fine, it detects invalid file file extensions, however if the file is over 2mb the script doesnt do what it should, and doesnt output any kind of result or do the upload.

Is this a memory issue on the server, or is it a issue with coding?

define('MAX_FILE_SIZE', 8388608); // 8 MB
$allowed_types = array('image/gif', 'image/pjpeg', 'image/jpeg', 'image/jpg', 'image/png', 'image/bmp');
$target_path = $uploadLocation . basename( $_FILES['file']['name']);
$extension = substr(strrchr($_FILES['file']['type'], "/"), 1);
    switch ($_FILES['file']['error']) {
        case UPLOAD_ERR_NO_FILE:
              $error = 'No file selected to upload!';
              echo $error;
              break;

        case UPLOAD_ERR_OK:
            if ($_FILES['file']['size'] > MAX_FILE_SIZE) {
                $error = 'File is too large!';
                echo $error;
                break;
            }
            if (!in_array($_FILES['file']['type'], $allowed_types)) {
                $error = 'That file type is not allowed!';
                echo $error;
                break;
            }
            if (move_uploaded_file($_FILES['file']['tmp_name'], $target_path)) {
                echo "The file: ".  basename( $_FILES['file']['name']). " has been uploaded!";  }
                else {
                $error = 'Could not move file, upload failed!';
                echo $error;
                break;  }
} 

[Jake Arkinstall]

It's because of many different settings in PHP.ini and httpd.conf - there is usually a server upload limit of 2mb.

Do you have access to php.ini and httpd.conf? Note that if you upload the files later to a server, you may have problems.

[galen]

default max execution time is 30 seconds. if the upload takes longer than 30 seconds it will fail.

[tdob]

It's because of many different settings in PHP.ini and httpd.conf - there is usually a server upload limit of 2mb.

Do you have access to php.ini and httpd.conf? Note that if you upload the files later to a server, you may have problems.

Your right the upload limit is 2M in the php config file. Is there a way to return this as a error message instead of displaying nothing?

Also, when I attempt to upload 70MB file to test (my limit is 8M so im expecting a error from the script) the script craps out, does PHP load the file into memory first even before it transfers it?

[Jake Arkinstall]

yup:

define('MAX_FILE_SIZE', 2097152); 

Sets the max size to 2mb.

[tdob]

yup:

define('MAX_FILE_SIZE', 2097152); 

Sets the max size to 2mb.

I know that, wondering if I could output that it was a server error as opposed to a script error... Also, that is of no help because if I try to upload a 25mb file, the script works for a while, then just does nothing, I dont even get an error message stating the file is too large.

Its as if PHP attempts to digest the entire file and ultimately craps out not letting the script do its thing.

[galen]

Read this and this

[cranial-bore]

HTTP isn't all that reliable for uploading large files. Even if you get everything configured right I think it's still going to be hit and miss with files more that a few MB

[tdob]

HTTP isn't all that reliable for uploading large files. Even if you get everything configured right I think it's still going to be hit and miss with files more that a few MB

The thing is, I dont want to upload large files. Just .pdf files or image files.

The limit on the PHP Server is 2mb, which is fine. If a user tries to upload a file up to 7mb they get the appropiate error based on the PHP limit.

My issue is, if a user tries (for whatever reason) to upload a larger file, nothing happens and the page kind of reloads itself seemingly ignoring all code/error checking. I suppose this is just something you have to deal with PHP/Uploads?

[Jake Arkinstall]

Shouldn't the class you use deal with files larger than MAX_FILE_SIZE?

[tdob]

Shouldn't the class you use deal with files larger than MAX_FILE_SIZE?

Im not 100% sure on what you mean. I have the hidden form field:

<input type="hidden" name="MAX_FILE_SIZE" value="1097152">

However, im not sure where it comes into play? Is it supposed to output an error?

[php_daemon]

Im not 100% sure on what you mean. I have the hidden form field:

<input type="hidden" name="MAX_FILE_SIZE" value="1097152">

However, im not sure where it comes into play? Is it supposed to output an error?

That should yeld a message on browser side. If the browser does that is another question.

If file size exceeds this value, a respective error code will be set to $_FILES['userfile']['error'] and upload will be considered as failed.

If you use this html field attribute, you must handle the error in php too.

[Jake Arkinstall]

No, i mean in the class, it checks for the file size:

if($_FILES['file']['size'] > MAX_FILE_SIZE) 

so it should error...