Как исправить проблему:Файлы не загружаются (POST запрос)?

Question

[Аркадий Астанин]

Здравствуйте, у меня тут возникла ошибка при загрузки файла.
OS: windows server 2019
WEB: nginx
PHP: 7.4.15

когда этот скрипт был на linux то нормально работал, но пришлось перейти на windows и перестал нормально работать.

PHP:

$expensions= array("mp4", "mkv");
	$file_ext=strtolower(end(explode('.',$_FILES['userfile']['name'])));
	if(in_array($file_ext,$expensions)=== false){
		echo "Расширение запрещено, выберите файл MP4 или MKV!\n";
	}else{
		
		$uploaddir = 'D:/nginx/www/video/source/'.$aData['name'].'/';
		
		if(is_uploaded_file($_FILES['userfile']['tmp_name'])){
			echo "\nDownloaded.\n";
		} else {
			echo "\nError of download!\n";
		}
		
		if(!is_dir($uploaddir)){
			mkdir($uploaddir, 0777, TRUE);
			@chmod($uploaddir, 0777);
		}
		
		$uploadfile = $uploaddir . basename(preg_replace('/ /u', '_', $_FILES['userfile']['name']));
		
		if (move_uploaded_file($_FILES['userfile']['tmp_name'], $uploadfile)) {
			echo "Файл корректен и был успешно загружен.\n";
		} else {
			echo "Возможная атака с помощью файловой загрузки!\n";
		}
	}

HTML:

<form action="input.php" method="post" class="d-uploader" id="form" enctype="multipart/form-data">
	<div class="lk-inp" style="display: grid;">
		<label for="name">Название релиза:</label>
		<input name="name" id="name" placeholder="Введите название релиза" type="text">
		<label for="userfile">Видео (mp4 и mkv):</label>
				
		<div class="form-group inputDnD">
			<input type="file" class="form-control-file" name="userfile" id="userfile" onchange="readUrl(this)" data-title="Drag and drop a file">
		</div>
	</div>
	
    <button class="btn btn-green" onclick="">Загрузить на сервер</button>
</form>

Ответ сервера всегда :

Error of download! Возможная атака с помощью файловой загрузки!

Вывод print_r($_FILES):

Array ( 
  [userfile] => Array (
      [name] => trailer.mp4 
      [type] => 
      [tmp_name] => 
      [error] => 1 
      [size] => 0 
   )
)

ТУТ Проверять

Comments

[FanatPHP]

Включи отображение ошибок

Answer 1

[Adamos]

if (move_uploaded_file($_FILES['userfile']['tmp_name'], $uploadfile)) {
      echo "Файл корректен и был успешно загружен.\n";
    } else if($_FILES['userfile']['error'] === UPLOAD_ERR_FORM_SIZE) {
      echo "А кто будет настраивать сервер под файло таких размеров?\n";
    }

Comments

[FanatPHP]

ну это кривизна же.
если в $_FILES['userfile']['tmp_name'] пусто, то move_uploaded_file будет ругаться.
проверять корректность надо как бы ДО попытки переместить файл, а не после

а главное это же не единственная возможная ошибка
я ставлю на то что у него в пути колбаса

[Adamos]

FanatPHP, пустота tmp_name проверяется выше по коду. Если при ошибке там должно быть пусто - значит, дело действительно в чем-то другом.

[FanatPHP]

Adamos, ну тем более - если проверяется, то в $_FILES['userfile']['error'] будет пусто, и эта проверка получается вообще бессмысленной

[Аркадий Астанин]

FanatPHP, $_FILES['userfile']['error'] - пустой, $_FILES['userfile']['tmp_name'] -пустой. Всё остальное нормально

[Adamos]

Аркадий Астанин, при пустом tmp_name ты должен получать "Error of download!"
Ну, и заменить пробелы на подчеркивания в названии недостаточно. В имени теоретически могут быть +:\/*.

[Аркадий Астанин]

Adamos, Вывод print_r($_FILES):
Array (
[userfile] => Array (
[name] => trailer.mp4
[type] =>
[tmp_name] =>
[error] => 1
[size] => 0
)
)

[FanatPHP]

ну и где тут $_FILES['userfile']['error'] - "пустой"?

[Adamos]

Аркадий Астанин, ну да, ошибка не вторая - больше, чем обещано - а первая - больше, чем разрешено сервером. См. вопрос в ответе ;)

[Аркадий Астанин]

FanatPHP, Прощу прощение, не заметил

[Аркадий Астанин]

Adamos, не понял что нужно

[Аркадий Астанин]

у меня стоит:

ini_set("max_execution_time", 300);
ini_set("upload_max_filesize", "1024M");
ini_set("post_max_size", "1024M");

размер файла - 7,05 МБ

[Adamos]

Аркадий Астанин, этими настройками можно подтереться.
Ini_set отрабатывает уже после того, как сервер получил post-запрос и рубанул файло по размеру.

[Аркадий Астанин]

Adamos, тогда скажи где редактировать

[Adamos]

Аркадий Астанин, сам-то как думаешь?

[Аркадий Астанин]

Adamos, Писали что php.ini, но у меня нету, есть только php.ini-development и php.ini-production другого нету

[Adamos]

Аркадий Астанин, значит, РНР из астрала настройки берет. Вот туда и пиши.
Ну серьезно, детский сад же.

[Аркадий Астанин]

Adamos,

[Adamos]

Аркадий Астанин, в конце списка аж два ридми. Не пробовал?

[Аркадий Астанин]

Adamos, пробовал

[Adamos]

Аркадий Астанин, а выделить текст "php.ini-production", нажать "искать в Гугле" и обнаружить десять ответов в первых десяти выдачах, причем в тройке - здесь же, на Тостере?

Answer 2

[no_one_safe]

А ты уверен что этот захардкоженный путь существует?
$uploaddir = 'D:/nginx/www/video/source

Comments

[Аркадий Астанин]

да

[Adamos]

Если даже не существует - в коде есть mkdir.
Он не будет работать, только если веб-серверу запрещено туда писать или вовсе нет диска D.