Bug #61463

_processed_ folder not outside file storage root possible

Added by Bernd Wilke about 5 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
File Abstraction Layer (FAL)
Target version:
-
Start date:
2014-09-09
Due date:
% Done:

100%

TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
hard
Is Regression:
No
Sprint Focus:

Description

I have a file storage for a readonly filemount.
as it is not possible to set the processed folder outside the root of a file storage1 I need to insert another local folder to enable a local (and writable) processed folder2.
as the files from that filemount are accessed from another server this additional folder is superflous.

[1] all entries of .. inside the path to the processed-folder result in an fatal error:
#1320286857: File ../_processed_ is not valid (".." and "//" is not allowed in path). (More information)

[2] as the filemount is readonly there is no possibility to create the processed folder inside the root directory of the file storage (and also no processed files):

PHP Warning
PHP Warning: rename(/var/www/typo3temp/preview_nFDQil.jpg,/var/www/imgservermount/_processed_/preview_aeg-182232_0_0a048ffd3d.jpg): No such file or directory in /var/www/typo3_src-6.1.10/typo3/sysext/core/Classes/Resource/Driver/LocalDriver.php line 622

which results in errors for each 'preview-image' (as there is no preview image):

exec_INSERTquery
caller     TYPO3\CMS\Core\Database\DatabaseConnection::exec_INSERTquery
ERROR     Column 'originalfilesha1' cannot be null
lastBuiltQuery     INSERT INTO sys_file_processedfile (width,height,checksum,identifier,name,storage,task_type,configuration,original,originalfilesha1,tstamp,crdate) VALUES ('64','64','0a048ffd3d','/_processed_/preview_aeg-182232_0_0a048ffd3d.jpg','preview_aeg-182232_0_0a048ffd3d.jpg','6','Image.Preview','a:0:{}','89881',NULL,'1410248566','1410248566')
debug_backtrace     require(o3project/typo3_src-6.1.10/typo3/sysext/filelist/mod1/index.php),o3project/typo3_src-6.1.10/typo3/mod.php#41 // TYPO3\CMS\Filelist\Controller\FileListController->main#46 // TYPO3\CMS\Filelist\FileList->generateList#290 // TYPO3\CMS\Filelist\FileList->getTable#199 // TYPO3\CMS\Filelist\FileList->formatFileList#353 // TYPO3\CMS\Core\Resource\File->process#622 // TYPO3\CMS\Core\Resource\ResourceStorage->processFile#293 // TYPO3\CMS\Core\Resource\Service\FileProcessingService->processFile#1150 // TYPO3\CMS\Core\Resource\Service\FileProcessingService->process#103 // TYPO3\CMS\Core\Resource\ProcessedFileRepository->add#140 // TYPO3\CMS\Core\Database\DatabaseConnection->exec_INSERTquery#123 // TYPO3\CMS\Core\Database\DatabaseConnection->debug#198


Related issues

Related to TYPO3 Core - Bug #62196: FAL Exception with readonly Storages, as _processed_ folders can't be created Closed 2014-10-13
Related to TYPO3 Core - Bug #70481: Ensure processing folder within another storage is created if not existing Closed 2015-10-07

Associated revisions

Revision fab21d38 (diff)
Added by Helmut Hummel over 4 years ago

[BUGFIX] Allow processed folders in different storage

The processingfolder of a storage can now be a combined identifier.
This makes it possible to have the processed files outside of the
storage in case of a read-only storage for instance.

Releases: master, 6.2
Resolves: #61463
Change-Id: I4f0e187db2aede33be40f62df3bb9f63e9706d46
Reviewed-on: http://review.typo3.org/32921
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>

Revision fd0f56b2 (diff)
Added by Helmut Hummel over 4 years ago

[BUGFIX] Allow processed folders in different storage

The processingfolder of a storage can now be a combined identifier.
This makes it possible to have the processed files outside of the
storage in case of a read-only storage for instance.

Releases: master, 6.2
Resolves: #61463
Change-Id: I4f0e187db2aede33be40f62df3bb9f63e9706d46
Reviewed-on: http://review.typo3.org/37584
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>

History

#1 Updated by Frans Saris about 5 years ago

  • Category set to File Abstraction Layer (FAL)
  • Complexity set to hard

This is more or less a missing feature.

Maybe we van somehow tell the driver to use typo3temp or some other local folder to store the processed files. But I'm not sure yet about the impact of that.

#2 Updated by Frans Saris almost 5 years ago

  • TYPO3 Version changed from 6.1 to 6.2
  • Sprint Focus set to On Location Sprint

Helmut started a WIP that also would fix this https://review.typo3.org/#/c/32921/

#3 Updated by Frans Saris over 4 years ago

  • Status changed from New to In Progress
  • Assignee set to Frans Saris

#4 Updated by Gerrit Code Review over 4 years ago

  • Status changed from In Progress to Under Review

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32921

#5 Updated by Gerrit Code Review over 4 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32921

#6 Updated by Gerrit Code Review over 4 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32921

#7 Updated by Gerrit Code Review over 4 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32921

#8 Updated by Gerrit Code Review over 4 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32921

#9 Updated by Gerrit Code Review over 4 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32921

#10 Updated by Gerrit Code Review over 4 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32921

#11 Updated by Gerrit Code Review over 4 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32921

#12 Updated by Gerrit Code Review over 4 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32921

#13 Updated by Gerrit Code Review over 4 years ago

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37584

#14 Updated by Helmut Hummel over 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#15 Updated by Gerrit Code Review over 4 years ago

  • Status changed from Resolved to Under Review

Patch set 2 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37584

#16 Updated by Helmut Hummel over 4 years ago

  • Status changed from Under Review to Resolved

#17 Updated by Anja Leichsenring almost 4 years ago

  • Sprint Focus deleted (On Location Sprint)

#18 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF