X-Git-Url: https://vcs.fsf.org/?a=blobdiff_plain;f=src%2Fconfigtest.php;h=47dc461b8c88f4526a29e6da6ae0ec58f0dff5cf;hb=b28e73038621072dea0b1e73e39c3faa206011cd;hp=7377d0c4eeb12c6627a59b2f5949c450ad9b7f9f;hpb=c772f1259065d613b8ba478fb21b038f995bc705;p=squirrelmail.git diff --git a/src/configtest.php b/src/configtest.php index 7377d0c4..47dc461b 100644 --- a/src/configtest.php +++ b/src/configtest.php @@ -52,6 +52,11 @@ if (file_exists(SM_PATH . 'config/config.php')) { require(SM_PATH . 'functions/global.php'); require(SM_PATH . 'functions/strings.php'); +/** + * get_location starts session and must be run before output is started. + */ +$test_location = get_location(); + ?> @@ -111,7 +116,9 @@ if(!check_php_version(4,1,0)) { } echo $IND . 'PHP version ' . PHP_VERSION . ' OK. (You have: ' . phpversion() . ". Minimum: 4.1.0)
\n"; -if ((bool) ini_get('register_globals')) { +/* test for boolean false and any string that is not equal to 'off' */ +if ((bool) ini_get('register_globals') && + strtolower(ini_get('register_globals'))!='off') { do_err('You have register_globals turned on. This is not an error, but it CAN be a security hazard. Consider turning register_globals off.', false); } $php_exts = array('session','pcre'); @@ -237,7 +244,32 @@ if (isset($plugins[0])) { do_err('You have enabled the '.$plugin.' plugin, which causes problems with this version of SquirrelMail. Please check the ReleaseNotes or other documentation for more information.', false); } } - echo $IND . "Plugins OK.
\n"; + // load plugin functions + include_once(SM_PATH . 'functions/plugin.php'); + // turn on output buffering in order to prevent output of new lines + ob_start(); + foreach ($plugins as $name) { + use_plugin($name); + } + // get output and remove whitespace + $output = trim(ob_get_contents()); + ob_end_clean(); + // if plugins output more than newlines and spacing, stop script execution. + if (!empty($output)) { + $plugin_load_error = 'Some output is produced when plugins are loaded.' + .' Usually it means error. Output said: '.htmlspecialchars($output); + do_err($plugin_load_error); + } + /** + * Hook is added in 1.5.2. Plugins should print error message and return true + * if there is an error in plugin. + */ + $plugin_err = boolean_hook_function('configtest'); + if($plugin_err) { + do_err('Some plugin tests failed.'); + } else { + echo $IND . "Plugins OK.
\n"; + } } else { echo $IND . "Plugins are not enabled in config.
\n"; } @@ -269,7 +301,9 @@ if ( $squirrelmail_default_language != 'en_US' ) { echo $IND . "Default language OK.
\n"; } -echo $IND . "Base URL detected as: " . htmlspecialchars(get_location()) . "
\n"; +echo $IND . "Base URL detected as: " . htmlspecialchars($test_location) . + " (location base " . (empty($config_location_base) ? 'autodetected' : 'set to ' . + htmlspecialchars($config_location_base)."") . ")
\n"; /* check minimal requirements for other security options */ @@ -470,6 +504,27 @@ if($imap_auth_mech == 'login' && stristr($capline, 'LOGINDISABLED') !== FALSE) { 'in the SquirrelMail configuration.', FALSE); } +if (stristr($capline, 'XMAGICTRASH') !== false) { + $magic_trash = 'It looks like IMAP_MOVE_EXPUNGE_TO_TRASH option is turned on ' + .'in your Courier IMAP configuration. Courier does not provide tools that ' + .'allow to detect folder used for Trash or commands are not documented. ' + .'SquirrelMail can\'t detect special trash folder. SquirrelMail manages ' + .'all message deletion or move operations internally and ' + .'IMAP_MOVE_EXPUNGE_TO_TRASH option can cause errors in message and ' + .'folder management operations. Please turn off IMAP_MOVE_EXPUNGE_TO_TRASH ' + .'option in Courier imapd configuration.'; + do_err($magic_trash,false); +} + +/* add warning about IMAP delivery */ +if (stristr($capline, 'XCOURIEROUTBOX') !== false) { + $courier_outbox = 'OUTBOX setting is enabled in your Courier imapd ' + .'configuration. SquirrelMail uses standard SMTP protocol or sendmail ' + .'binary to send emails. Courier IMAP delivery method is not supported' + .' and can create duplicate email messages.'; + do_err($courier_outbox,false); +} + /** OK, close connection */ fputs($stream, "A004 LOGOUT\r\n"); fclose($stream); @@ -483,7 +538,7 @@ if (function_exists('gettext')) { /* optional setlocale() tests. Should work only on glibc systems. */ if (sqgetGlobalVar('testlocales',$testlocales,SQ_GET)) { - include_once(SM_PATH . 'functions/i18n.php'); + include_once(SM_PATH . 'include/languages.php'); echo $IND . $IND . 'Testing translations:
'; foreach ($languages as $lang_code => $lang_data) { /* don't test aliases */ @@ -556,9 +611,9 @@ if (function_exists('iconv')) { } else { echo "Iconv functions are unavailable.
\n"; } -// same test as in include/validate.php +// same test as in include/init.php + date_default_timezone_set check echo "$IND timezone - "; -if ( (!ini_get('safe_mode')) || +if ( (!ini_get('safe_mode')) || function_exists('date_default_timezone_set') || !strcmp(ini_get('safe_mode_allowed_env_vars'),'') || preg_match('/^([\w_]+,)*TZ/', ini_get('safe_mode_allowed_env_vars')) ) { echo "Webmail users can change their time zone settings. \n";