Project

General

Profile

Actions

Bug #20069

closed

Misinterpretaion of php.ini settings when no limit is set

Added by Jo Schulze about 15 years ago. Updated over 6 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 1 (0 open1 closed)

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

Actions
Actions #1

Updated by Oliver Hader about 15 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... ;-)

Actions #2

Updated by Christian Kuhn about 15 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

Actions #3

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

Actions #4

Updated by Gerrit Code Review over 12 years ago

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

Actions #5

Updated by Gerrit Code Review about 12 years ago

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

Actions #6

Updated by Gerrit Code Review about 12 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

Actions #7

Updated by Gerrit Code Review about 12 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

Actions #8

Updated by Ernesto Baschny about 12 years ago

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

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF