Bug #81024

getFilenameFromMainDir() triggers PHP Warnings open_basedir

Added by Christian Toffolo over 3 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Christian Toffolo
Category:
Backend API
Target version:
8 LTS
Start date:
2016-11-28
Due date:
2016-11-28
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.1
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Method

\TYPO3\CMS\Core\Resource\ResourceCompressor::getFilenameFromMainDir()

triggers warnings
core: Core: Error handler (BE): PHP Warning: is_file(): open_basedir restriction in effect. File(/var/www/website/htdocs/../typo3conf/ext/extension/Resources/Public/file) is not within the allowed path(s):

when the BE is browsed.

This happens because "is_file($absolutePath)" checks outside the website root in the case of a passed file that has a relative path that begins with "../typo3conf/extension..." and open_basedir doesn't allow it.

Me and Markus Klein talked about the problem and decided to use the PHP error control operator @ to suppress the warnings because other solutions could break the code in other cases.

Related issues

Has duplicate TYPO3 Core - Bug #81115: TYPO3-Exception in BE-Log due to "PHP Warning: is_file(): open_basedir restriction in effect"Closed2016-11-282016-11-28

Actions
Is duplicate of TYPO3 Core - Bug #79218: ResourceCompressor.php PHP Warning: is_file(): open_basedir restriction in effectClosed2017-01-09

Actions
Follows TYPO3 Core - Bug #78803: ResourceCompressor can't deal with paths relative to docrootClosedMarkus Klein2016-11-25

Actions
#1

Updated by Gerrit Code Review over 3 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/52614

#2

Updated by Gerrit Code Review over 3 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/52614

#3

Updated by Markus Klein over 3 years ago

  • Has duplicate Bug #81115: TYPO3-Exception in BE-Log due to "PHP Warning: is_file(): open_basedir restriction in effect" added
#4

Updated by Gerrit Code Review over 3 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/52614

#5

Updated by Stephan Großberndt over 3 years ago

  • Due date set to 2016-11-28
  • Start date changed from 2017-04-26 to 2016-11-28
  • Follows Bug #78803: ResourceCompressor can't deal with paths relative to docroot added
#6

Updated by Markus Klein over 3 years ago

  • Is duplicate of Bug #79218: ResourceCompressor.php PHP Warning: is_file(): open_basedir restriction in effect added
#7

Updated by Gerrit Code Review over 3 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/52694

#8

Updated by Christian Toffolo over 3 years ago

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

Updated by Riccardo De Contardi about 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF