Feature #69975

Make typo3temp-path configurable

Added by Jonas Enders about 5 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
System/Bootstrap/Configuration
Start date:
2015-09-19
Due date:
% Done:

0%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

The typo3temp-path should be configurable.

I would suggest:

$GLOBALS['TYPO3_CONF_VARS']['BE']['tempDir']

I'm just trying to get TYPO3 frontend to run on Google Application Engine and this setting would be very helpful.


Related issues

Related to TYPO3 Core - Bug #81740: temp directory name shouldn't be hardcoded in ClearCacheServiceClosed2017-06-29

Actions
#1

Updated by Christian Kuhn about 5 years ago

I'm not a big fan of this idea. This will trigger tons of changes and the benefit looks rather low. This also affects single extension authors and if those hard code typo3temp/ again, you're still doomed. Additionally, we may easily run into funny chicken-egg problems within bootstrap if we un-hardcode this, since the bootstrap tends to write stuff to typo3temp/ before configuration is loaded.

When I last run into issues in this area, it became obvious to me that we must change typo3temp in a way that no single file is stored within typo3temp/ directly, but everything is put into sub directories. This would give us a chance in cloud to have parts of typo3temp "shared" and other parts "locally" on nodes. We need that, since some stuff actually must stay locally, while other stuff needs to be shared. And this can only be reached if there is no single file in typo3temp/ itself.

Resource management together with work on typo3temp content will hopefully be a topic for cms 8, since we also need this if we finally want to extract extensions out of web root.

Imho, making typo3temp/ directory name configurable does not really help in the process to make the system better cloud friendly. Could you elaborate a bit what exactly is your problem with typo3temp/ in google app engine?

#2

Updated by Jonas Enders about 5 years ago

I started to make the typo3temp path configurable and run into some problems which makes my suggested change not useful for my approach. My idea was to replace the path with the stream wrapper "gs://bucket_name/", because the normal filesystem in GAE is read-only. But in most cases the path is accessed absolutely (eg "PATH_site . $this->targetDirectory"). Changing "$this->targetDirectory" in this example to "gs://bucket_name/" wouldn't help anyway.

In the issue of backward compatibility of extensions I see unfortunately no possibility at all.

So, if you have a good idea to reach the goal, i would be happy to put some work on this project.

#3

Updated by Benni Mack almost 5 years ago

  • Category set to Miscellaneous
  • Assignee set to Benni Mack
  • Target version set to 8.0
#4

Updated by Benni Mack almost 5 years ago

Hey Jonas,

in the last 3 weeks we managed to move everything to typo3temp/var/ or typo3temp/assets/ which can now be mounted on cloud hosting easily. Now these two parameters should become configurable, but I don't think a TYPO3_CONF_VARS option is the right way to do so. Either a constant (to avoid the PATH_site . 'typo3temp/var/' thing to move that outside of the directory, or we look into puli (http://docs.puli.io/en/latest/) to become even more flexible. Ping me on slack to get this started...

#5

Updated by Benni Mack almost 5 years ago

  • Status changed from New to Accepted
#6

Updated by Gerrit Code Review over 4 years ago

  • Status changed from Accepted to Under Review

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/46419

#7

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 https://review.typo3.org/46419

#8

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 https://review.typo3.org/46419

#9

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 https://review.typo3.org/46419

#10

Updated by Gerrit Code Review over 4 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/47805

#11

Updated by Benni Mack over 4 years ago

  • Target version changed from 8.0 to 8.2
#12

Updated by Benni Mack over 4 years ago

  • Target version changed from 8.2 to 8.3
#13

Updated by Benni Mack about 4 years ago

  • Target version changed from 8.3 to Candidate for Major Version
#14

Updated by Susanne Moog about 3 years ago

  • Category changed from Miscellaneous to System/Bootstrap/Configuration
#15

Updated by Benni Mack about 3 years ago

  • Status changed from Under Review to Accepted
#16

Updated by Benni Mack over 2 years ago

  • Status changed from Accepted to Resolved

Going with TYPO3 v9, you now have the possibility to move typo3temp somewhere else.

#17

Updated by Riccardo De Contardi over 2 years ago

  • Related to Bug #81740: temp directory name shouldn't be hardcoded in ClearCacheService added
#18

Updated by Benni Mack about 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF