Bug #84178
closedCannot create but upload file with "@" in name
100%
Description
In FAL there are at least two different ways to create files which apparently do not apply the same sanitation/validation rules to file names.
This can be verified easily in the Filelist module: if you try to upload a file called foo@bar.txt everything simply works.
But if you create a file called foo@bar.txt a ResourceDoesNotExistException
is thrown:
#1329647780: Object with identifier "1:/foo@bar.txt" does not exist in storage
When opening the file list once more after this, an error flash message is shown which says File name "foo@bar.txt" was not allowed!.
This behavior can be traced back to these two code paths:
LocalDriver::addFile()
callsLocalDriver::sanitizeFileName()
which accepts foo@bar.txt (this is used e.g. for file uploads)LocalDriver::createFile()
callsAbstractDriver::isValidFilename()
which denies foo@bar.txt (this is used for everything else)
Updated by Gerrit Code Review over 6 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56089
Updated by Gerrit Code Review over 6 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56089
Updated by Wolfgang Klinger over 6 years ago
- Related to Bug #84079: Error after creating file containing spaces added
Updated by Gerrit Code Review over 6 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56089
Updated by Gerrit Code Review over 6 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56089
Updated by Gerrit Code Review over 6 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56089
Updated by Gerrit Code Review over 6 years ago
Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56352
Updated by Wolfgang Klinger over 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset cb905282b1cbc6e222697b1872d3efa987f800b5.