11. Dezember 2013

ExtBase File Upload vor 6.1 (4.5, 4.7, 6.0) die 2te.

in Dev, Extension, TYPO3

Ich hab schonmal hier: https://typo3blogger.de/extbase-file-upload/ etwas zu dazu geschrieben.

Da das allerdings ab und zu in Foren nochmal Thema ist und der alte Artikel mit der heißen Nadel gestrickt war, gibt’s noch nen kleines Followup:

Code sagt mehr als tausend Worte:
Usage im Controller:

   if ($uploadedFile = $this->handleFileUpload('fooBarFile', 'uploads/tx_hasteNichtGesehen') {
    ...was auch immer
   }

Controller Code

  /**
   * Forwards back to referrer with setting an error
   *
   * @param $fieldName
   * @param $errorMessage
   * @param $errorCode
   * @return void
   */
  protected function throwBackWithError($fieldName, $errorMessage, $errorCode) {
    $error = $this->objectManager->get('Tx_Extbase_MVC_Controller_ArgumentError' , $fieldName);
    $error->addErrors(array(
        $this->objectManager->get('Tx_Extbase_Validation_Error', $errorMessage,$errorCode)
      )
    );
 
    $this->request->setErrors(array($error));
    $referrer = $this->request->getInternalArgument('__referrer');
    $this->forward($referrer['actionName'], $referrer['controllerName'], $referrer['extensionName'], $this->request->getArguments());
  }
 
  /**
   * Handles file upload
   *
   * @param $fieldName
   * @param $uploadFolder
   * @return bool|string
   */
  protected function handleFileUpload($fieldName, $uploadFolder) {
    $pluginNamespace = $this->objectManager->get('Tx_Extbase_Service_ExtensionService')
      ->getPluginNamespace($this->request->getControllerExtensionName(), $this->request->getPluginName());
 
    if (!$_FILES ||
      !isset($_FILES[$pluginNamespace]['name'][$fieldName]) ||
      $_FILES[$pluginNamespace]['error'][$fieldName] === UPLOAD_ERR_NO_FILE) {
 
      return FALSE;
    }
 
    if ($_FILES[$pluginNamespace]['error'][$fieldName] === UPLOAD_ERR_FORM_SIZE ||
      $_FILES[$pluginNamespace]['error'][$fieldName] === UPLOAD_ERR_INI_SIZE) {
 
      $this->throwBackWithError($fieldName, 'File upload size exceeded', 1386779047);
    }
 
    $uniqueFileName = $this->objectManager->get('t3lib_basicFileFunctions')->getUniqueName(
      $_FILES[$pluginNamespace]['name'][$fieldName],
      t3lib_div::getFileAbsFileName($uploadFolder)
    );
 
    t3lib_div::upload_copy_move(
      $_FILES[$pluginNamespace]['tmp_name'][$fieldName],
      $uniqueFileName
    );
 
    return basename($uniqueFileName);
  }

Für die Größenbeschränkung ist noch folgendes sinnvoll:

<input type="hidden" name="MAX_FILE_SIZE" value="1048576" />
<f:form.upload name="fooBarFile" />

Mehr zum „MAX_FILE_SIZE“ Feld gibts hier:
http://php.net/manual/de/features.file-upload.post-method.php
Und zu den Errorcodes hier:
http://www.php.net/manual/en/features.file-upload.errors.php

Das ist natürlich nicht die reine Lehre aber ein pragmatisch, gangbarer Ansatz.

Das ganze funktioniert nur bis TYPO3 6.1, weil dort das $_FILES Array durch ExtBase umgeschrieben wird. Ich hatte bisher noch nicht damit zu tun, aber sobald das der Fall ist gibt’s nen Artikel dazu.

Nachtrag 12.12.2013:
Wie in den Kommentaren schon angemerkt ist es natürlich sinnvoll sowohl file type als auch die Größe nochmal serverseitig zu validieren. Das kann dann so aussehen:

/**
 + Handles file upload
 *
 * @param $fieldName
 * @param $uploadFolder
 * @param int $maxFileSize
 * @param array $allowedFileTypes
 * @return bool|string
 */
protected function handleFileUpload($fieldName, $uploadFolder, 	$maxFileSize=1048576,
	$allowedFileTypes = array('application/pdf')) {
 
	$pluginNamespace = $this->objectManager->get('Tx_Extbase_Service_ExtensionService')
		->getPluginNamespace($this->request->getControllerExtensionName(), $this->request->getPluginName());
 
	if (!$_FILES ||
		!isset($_FILES[$pluginNamespace]['name'][$fieldName]) ||
		$_FILES[$pluginNamespace]['error'][$fieldName] === UPLOAD_ERR_NO_FILE) {
 
		return FALSE;
	}
 
	if ($_FILES[$pluginNamespace]['error'][$fieldName] === UPLOAD_ERR_FORM_SIZE ||
		$_FILES[$pluginNamespace]['error'][$fieldName] === UPLOAD_ERR_INI_SIZE ||
		$_FILES[$pluginNamespace]['size'][$fieldName] > $maxFileSize) {
 
		$this->throwBackWithError($fieldName, 'File upload size exceeded', 1386779047);
	}
 
	if (!in_array(mime_content_type($_FILES[$pluginNamespace]['tmp_name'][$fieldName]), $allowedFileTypes)) {
		$this->throwBackWithError($fieldName, 'File type not allowed', 1386855251);
	}
 
	$uniqueFileName = $this->objectManager->get('t3lib_basicFileFunctions')->getUniqueName(
		$_FILES[$pluginNamespace]['name'][$fieldName],
		t3lib_div::getFileAbsFileName($uploadFolder)
	);
 
	t3lib_div::upload_copy_move(
		$_FILES[$pluginNamespace]['tmp_name'][$fieldName],
		$uniqueFileName
	);
 
	return basename($uniqueFileName);
}

9 Reaktionen zu “ExtBase File Upload vor 6.1 (4.5, 4.7, 6.0) die 2te.”

Kommentare abonnieren (RSS) oder TrackBack URL

Hallo,

Wie sieht es hier eigentlich mit der Sicherheit aus ? Wird durch Extbase automatisch geprüft, das z.B. keine PHP Dateien oder sonstige hochgeladen werden ?
Und gibt es für die Größenbeschränkung auch noch eine Überprüfung ? oder kann ich die angabe im inputFeld einfach im HTML Code überschreiben ?

sven am 12. Dezember 2013 um 09:15

Extbase prüft da gar nichts automatisch.
Zur Größe: klar die kann man einfach überschreiben Du hast aber die Möglichkeit $_FILES[$pluginNamespace][’size‘]$fieldName] ebenfalls zu prüfen.
Das gleiche gilt für den type. Entweder über: $_FILES[$pluginNamespace][‚type‘]$fieldName] oder den mimetype der Datei direkt.

Nikolas Hagelstein am 12. Dezember 2013 um 09:21

Hi,

danke für den Hinweis!
Da keine Überprüung zu sehen war, nahm ich an das intern zumindest irgendwo gegen das „fileDenyPattern“ geprüft wird. Wenn dem nicht so ist, könntest du das vielleicht in deinem Betirag erwähnen ? Ich vermute das sonst der ein oder andere über google hierher kommt Copy&Paste macht und sich damit eventuell ne schöne Sicherheitslücke einbaut.

sven am 12. Dezember 2013 um 09:41

Ja, das ist richtig. Ich hab das nochmal dazugefügt.

Nikolas Hagelstein am 12. Dezember 2013 um 14:54

Hallo Leute

ich versuche diese file upload zu realisieren.

mein problem ist ich nutze typo3 in der version 6.1.5

ich habe problem mit folgende zeile:

$uniqueFileName = $this->objectManager->get(‚t3lib_basicFileFunctions‘)->getUniqueName(
$_FILES[$pluginNamespace][’name‘][’newBewerbung‘][$fieldName],
TYPO3\CMS\Core\Utility\GeneralUtility::getFileAbsFileName($uploadFolder)
);

ich musste feststellen dass in der version 6.1.5 zb t3lib_div veraltet ist, das heisst der aufruft:

t3lib_div::getFileAbsFileName($uploadFolder geht nicht!
deswegen musste ich so verändern:
TYPO3\CMS\Core\Utility\GeneralUtility::getFileAbsFileName($uploadFolder)

trotzdem funktioniert es nicht. ich frage mich ob t3lib_basicFileFunctions auch in der 6 version veraltet ist?
danke in voraus!

sardinas am 14. Dezember 2013 um 17:34

Hallo sardinas. Ja, ist auch veraltet, hat aber keinen Einfluss auf die Ausführung. Auch die „alten“ Klasse gehen in 6.1 noch… Das Problem muss wo anderes liegen… Grüße, Tim

Tim Lochmüller am 14. Dezember 2013 um 17:38

Hallo Danke

für die schnelle antwort!!!
ich bin sicher dass meine anwendung in folgende zeile aufhört:

$uniqueFileName = $this->objectManager->get(‚t3lib_basicFileFunctions‘)->getUniqueName(
$_FILES[$pluginNamespace][’name‘][’newBewerbung‘][$fieldName],
TYPO3CMSCoreUtilityGeneralUtility::getFileAbsFileName($uploadFolder)
);

meine frage: muss man in controller $this->objectManager initialisieren?

sardinas am 14. Dezember 2013 um 18:25

t3lib gibts nicht mehr:
TYPO3\CMS\Core\Utility\File\BasicFileUtility

das $_FILES array wird in 6.1 umgeschrieben

Nikolas Hagelstein am 14. Dezember 2013 um 18:33

Wobei der alte Aufruf auch gehen sollte

Nikolas Hagelstein am 14. Dezember 2013 um 18:37

Kategorien

Archiv