Bug #20069

Misinterpretaion of php.ini settings when no limit is set

Added by Jo Schulze over 13 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Should have
Category:
Install Tool
Target version:
Start date:
2009-02-21
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.2
PHP Version:
5.2
Tags:
Complexity:
Is Regression:
Sprint Focus:

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)


Related issues

Related to TYPO3 Core - Bug #32231: Bad check on memory limit reportingClosed2011-12-01

Actions
#1

Updated by Oliver Hader over 13 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... ;-)

#2

Updated by Christian Kuhn over 13 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

#3

Updated by Gerrit Code Review over 10 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

#4

Updated by Gerrit Code Review over 10 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7067

#5

Updated by Gerrit Code Review over 10 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7067

#6

Updated by Gerrit Code Review over 10 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

#7

Updated by Gerrit Code Review over 10 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

#8

Updated by Ernesto Baschny over 10 years ago

  • Status changed from Under Review to Resolved
  • Assignee set to Ernesto Baschny
  • Target version changed from 0 to 4.5.12
#9

Updated by Riccardo De Contardi almost 5 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF