Merge pull request #15617 from mmattel/getenv_system_check_and_warning
Checks if getenv returns proper system environment variable results
This commit is contained in:
commit
6d3f120d23
|
@ -123,6 +123,9 @@ $databaseOverload = (strpos(\OCP\Config::getSystemValue('dbtype'), 'sqlite') !==
|
||||||
$template->assign('databaseOverload', $databaseOverload);
|
$template->assign('databaseOverload', $databaseOverload);
|
||||||
$template->assign('cronErrors', $appConfig->getValue('core', 'cronErrors'));
|
$template->assign('cronErrors', $appConfig->getValue('core', 'cronErrors'));
|
||||||
|
|
||||||
|
// warn if php is not setup properly to get system variables with getenv
|
||||||
|
$template->assign('getenvServerNotWorking', empty(getenv('PATH')));
|
||||||
|
|
||||||
// warn if Windows is used
|
// warn if Windows is used
|
||||||
$template->assign('WindowsWarning', OC_Util::runningOnWindows());
|
$template->assign('WindowsWarning', OC_Util::runningOnWindows());
|
||||||
|
|
||||||
|
|
|
@ -67,6 +67,16 @@ if ($_['mail_smtpmode'] == 'qmail') {
|
||||||
<h2><?php p($l->t('Security & setup warnings'));?></h2>
|
<h2><?php p($l->t('Security & setup warnings'));?></h2>
|
||||||
<ul>
|
<ul>
|
||||||
<?php
|
<?php
|
||||||
|
// is php setup properly to query system environment variables like getenv('PATH')
|
||||||
|
if ($_['getenvServerNotWorking']) {
|
||||||
|
?>
|
||||||
|
<li>
|
||||||
|
<?php p($l->t('php does not seem to be setup properly to query system environment variables. The test with getenv("PATH") only returns an empty response.')); ?><br>
|
||||||
|
<?php p($l->t('Please check the installation documentation for php configuration notes and the php configuration of your server, especially when using php-fpm.')); ?>
|
||||||
|
</li>
|
||||||
|
<?php
|
||||||
|
}
|
||||||
|
|
||||||
// is read only config enabled
|
// is read only config enabled
|
||||||
if ($_['readOnlyConfigEnabled']) {
|
if ($_['readOnlyConfigEnabled']) {
|
||||||
?>
|
?>
|
||||||
|
|
Loading…
Reference in New Issue