Bug #88256: FileWriter creates directories like php: or file: when eg. php://stdout is used a logfile destination - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #88256

closed

FileWriter creates directories like php: or file: when eg. php://stdout is used a logfile destination

Added by Sven Carstens over 5 years ago. Updated over 4 years ago.

Status:

Closed

Priority:

Must have

Assignee:

Category:

Target version:

Start date:

2019-05-02

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

typo3/sysext/core/Classes/Log/Writer/FileWriter.php uses GeneralUtility::mkdir_deep to create probably missing directories for the configured logfile.
When using php://stdout as a logfile destination empty php: directories popup under the web root and under typo3/.

Before creating the directories we have to check for the usage of a [https://www.php.net/manual/en/wrappers.php]

Actions

Copy link

Updated by Gerrit Code Review over 5 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/c/Packages/TYPO3.CMS/+/60643

Actions

Copy link

Updated by Gerrit Code Review over 5 years ago

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/c/Packages/TYPO3.CMS/+/60645

Actions

Copy link

Updated by Gerrit Code Review over 5 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/c/Packages/TYPO3.CMS/+/60643

Actions

Copy link

Updated by Gerrit Code Review over 5 years ago

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/c/Packages/TYPO3.CMS/+/60646

Actions

Copy link

Updated by Gerrit Code Review over 5 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/c/Packages/TYPO3.CMS/+/60646

Actions

Copy link

Updated by Gerrit Code Review over 5 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/c/Packages/TYPO3.CMS/+/60646

Actions

Copy link

Updated by Markus Klein over 5 years ago

Can you please describe you use case here as well? I suppose containers?

Actions

Copy link

Updated by Sven Carstens over 5 years ago

My use-case here is to set logging to php://stdout inside of containers.

Actions

Copy link

Updated by Gerrit Code Review almost 5 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/c/Packages/TYPO3.CMS/+/60646

Actions

Copy link

#10

Updated by Gerrit Code Review almost 5 years ago

Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62898

Actions

Copy link

#11