Bug #20069
closedMisinterpretaion of php.ini settings when no limit is set
0%
Description
After 1-2-3 installation the setup summary report
/install/index.php?TYPO3_INSTALL[type]=config
reports warnings about settings which in fact are perfctly ok:
memory_limit = -1 (no limit concerning memory) is reported as "Memory limit below 16MB"
max_execution_time = 0 (no limit concerning execution time) is reported as "Maximum execution time below 30 seconds"
(issue imported from #M10512)
Updated by Oliver Hader almost 16 years ago
Hm... in general I don't think that setting no limits is a good idea. Scripts with memory leaks might run towards the moon until it's year 3000... ;-)
Updated by Christian Kuhn almost 16 years ago
I agree, its probably not a good idea to disable those limits.
But still, both values are documented as valid php.ini settings and I think the install tool should reflect this. Maybe a warning that these settings are unusual and might harm the system is a good idea.
http://de2.php.net/manual/en/info.configuration.php#ini.max-execution-time
http://de2.php.net/manual/en/ini.core.php#ini.memory-limit
Updated by Gerrit Code Review almost 13 years ago
- Status changed from New to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7067
Updated by Gerrit Code Review almost 13 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7067
Updated by Gerrit Code Review almost 13 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7067
Updated by Gerrit Code Review almost 13 years ago
Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/8709
Updated by Gerrit Code Review almost 13 years ago
Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/8710
Updated by Ernesto Baschny almost 13 years ago
- Status changed from Under Review to Resolved
- Assignee set to Ernesto Baschny
- Target version changed from 0 to 4.5.12
Updated by Riccardo De Contardi about 7 years ago
- Status changed from Resolved to Closed