Skip to content

Commit 3939923

Browse files
committed
Merge pull request silverstripe#188 from chillu/install-display-errors-check
MINOR Check for display_errors=Off during installation (fixes silverstripe#5779)
2 parents 81484d5 + fcf103c commit 3939923

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

dev/install/install.php5

+1
Original file line numberDiff line numberDiff line change
@@ -398,6 +398,7 @@ class InstallRequirements {
398398

399399
$this->suggestPHPSetting('asp_tags', array(''), array('PHP Configuration', 'asp_tags option turned off', 'This should be turned off as it can cause issues with SilverStripe'));
400400
$this->suggestPHPSetting('magic_quotes_gpc', array(''), array('PHP Configuration', 'magic_quotes_gpc option turned off', 'This should be turned off, as it can cause issues with cookies. More specifically, unserializing data stored in cookies.'));
401+
$this->suggestPHPSetting('display_errors', array(''), array('PHP Configuration', 'display_errors option turned off', 'This should be turned off, as it can expose sensitive data to website users.'));
401402

402403
// Check memory allocation
403404
$this->requireMemory(32*1024*1024, 64*1024*1024, array("PHP Configuration", "Memory allocated (PHP config option 'memory_limit')", "SilverStripe needs a minimum of 32M allocated to PHP, but recommends 64M.", ini_get("memory_limit")));

0 commit comments

Comments
 (0)