From 09e8bdfd18f81e432c5f392e398b8c8088168721 Mon Sep 17 00:00:00 2001 From: Andrew Dolgov Date: Tue, 13 Dec 2011 19:20:26 +0400 Subject: [PATCH] simplify default global config, expand sanity_check messages --- classes/feeds.php | 10 +- classes/pref_users.php | 14 +-- classes/rpc.php | 12 +- config.php-dist | 225 +++++++++++++++++++------------------ include/db.php | 4 +- include/functions.php | 28 ++--- include/rssfuncs.php | 8 +- include/sanity_check.php | 226 ++++++++++++++++---------------------- include/sanity_config.php | 6 +- include/sessions.php | 2 +- register.php | 24 ++-- utility.css | 1 - 12 files changed, 266 insertions(+), 294 deletions(-) diff --git a/classes/feeds.php b/classes/feeds.php index 98ea55c0..5ecac494 100644 --- a/classes/feeds.php +++ b/classes/feeds.php @@ -715,12 +715,10 @@ class Feeds extends Protected_Handler { onclick=\"editArticleNote($id)\" alt='PubNote' title='".__('Edit article note')."'>"; - if (DIGEST_ENABLE) { - $reply['content'] .= "link, 'images/art-email.png')."\" - style=\"cursor : pointer\" - onclick=\"emailArticle($id)\" - alt='Zoom' title='".__('Forward by email')."'>"; - } + $reply['content'] .= "link, 'images/art-email.png')."\" + style=\"cursor : pointer\" + onclick=\"emailArticle($id)\" + alt='Zoom' title='".__('Forward by email')."'>"; if (ENABLE_TWEET_BUTTON) { $reply['content'] .= "link, 'images/art-tweet.png')."\" diff --git a/classes/pref_users.php b/classes/pref_users.php index 63e1fd22..b9d162fd 100644 --- a/classes/pref_users.php +++ b/classes/pref_users.php @@ -308,16 +308,16 @@ class Pref_Users extends Protected_Handler { $mail->CharSet = "UTF-8"; - $mail->From = DIGEST_FROM_ADDRESS; - $mail->FromName = DIGEST_FROM_NAME; + $mail->From = SMTP_FROM_ADDRESS; + $mail->FromName = SMTP_FROM_NAME; $mail->AddAddress($email, $login); - if (DIGEST_SMTP_HOST) { - $mail->Host = DIGEST_SMTP_HOST; + if (SMTP_HOST) { + $mail->Host = SMTP_HOST; $mail->Mailer = "smtp"; - $mail->SMTPAuth = DIGEST_SMTP_LOGIN != ''; - $mail->Username = DIGEST_SMTP_LOGIN; - $mail->Password = DIGEST_SMTP_PASSWORD; + $mail->SMTPAuth = SMTP_LOGIN != ''; + $mail->Username = SMTP_LOGIN; + $mail->Password = SMTP_PASSWORD; } $mail->IsHTML(false); diff --git a/classes/rpc.php b/classes/rpc.php index c553b9b9..272c25ef 100644 --- a/classes/rpc.php +++ b/classes/rpc.php @@ -570,7 +570,7 @@ class RPC extends Protected_Handler { $reply = array(); - if (DIGEST_ENABLE && $_SESSION['email_secretkey'] && + if ($_SESSION['email_secretkey'] && $secretkey == $_SESSION['email_secretkey']) { $_SESSION['email_secretkey'] = ''; @@ -593,12 +593,12 @@ class RPC extends Protected_Handler { $mail->FromName = $fromname; $mail->AddAddress($destination); - if (DIGEST_SMTP_HOST) { - $mail->Host = DIGEST_SMTP_HOST; + if (SMTP_HOST) { + $mail->Host = SMTP_HOST; $mail->Mailer = "smtp"; - $mail->SMTPAuth = DIGEST_SMTP_LOGIN != ''; - $mail->Username = DIGEST_SMTP_LOGIN; - $mail->Password = DIGEST_SMTP_PASSWORD; + $mail->SMTPAuth = SMTP_LOGIN != ''; + $mail->Username = SMTP_LOGIN; + $mail->Password = SMTP_PASSWORD; } $mail->IsHTML(false); diff --git a/config.php-dist b/config.php-dist index 4e231a6f..e4b5ae32 100644 --- a/config.php-dist +++ b/config.php-dist @@ -1,5 +1,8 @@ CharSet = "UTF-8"; - $mail->From = DIGEST_FROM_ADDRESS; - $mail->FromName = DIGEST_FROM_NAME; + $mail->From = SMTP_FROM_ADDRESS; + $mail->FromName = SMTP_FROM_NAME; $mail->AddAddress($line["email"], $line["login"]); - if (DIGEST_SMTP_HOST) { - $mail->Host = DIGEST_SMTP_HOST; + if (SMTP_HOST) { + $mail->Host = SMTP_HOST; $mail->Mailer = "smtp"; - $mail->SMTPAuth = DIGEST_SMTP_LOGIN != ''; - $mail->Username = DIGEST_SMTP_LOGIN; - $mail->Password = DIGEST_SMTP_PASSWORD; + $mail->SMTPAuth = SMTP_LOGIN != ''; + $mail->Username = SMTP_LOGIN; + $mail->Password = SMTP_PASSWORD; } $mail->IsHTML(true); @@ -3288,12 +3286,10 @@ onclick=\"editArticleNote($id)\" alt='PubNote' title='".__('Edit article note')."'>"; - if (DIGEST_ENABLE) { - $rv['content'] .= "Zoom"; - } + $rv['content'] .= "Zoom"; if (ENABLE_TWEET_BUTTON) { $rv['content'] .= " 0) { + if (!SINGLE_USER_MODE && DAEMON_UPDATE_LOGIN_LIMIT > 0) { if (DB_TYPE == "pgsql") { $login_thresh_qpart = "AND ttrss_users.last_login >= NOW() - INTERVAL '".DAEMON_UPDATE_LOGIN_LIMIT." days'"; } else { @@ -143,7 +147,7 @@ } // Send feed digests by email if needed. - if (DAEMON_SENDS_DIGESTS) send_headlines_digests($link); + send_headlines_digests($link); } // function update_daemon_common diff --git a/include/sanity_check.php b/include/sanity_check.php index 2769b4b5..aebe4f89 100644 --- a/include/sanity_check.php +++ b/include/sanity_check.php @@ -2,174 +2,134 @@ require_once "functions.php"; if (!file_exists("config.php")) { - print "Fatal Error: You forgot to copy - config.php-dist to config.php and edit it.\n"; - exit; - } - - define('EXPECTED_CONFIG_VERSION', 23); - define('SCHEMA_VERSION', 86); - - require_once "config.php"; - require_once "sanity_config.php"; - - if (CONFIG_VERSION != EXPECTED_CONFIG_VERSION) { - $err_msg = "config: your config file version is incorrect. See config.php-dist.\n"; - } - - $purifier_cache_dir = CACHE_DIR . "/htmlpurifier"; + $err_msg = "Configuration file not found. Looks like you forgot to copy config.php-dist to config.php and edit it."; + } else { - if (!is_writable($purifier_cache_dir)) { - $err_msg = "config: HTMLPurifier cache directory should be writable by anyone (chmod -R 777 $purifier_cache_dir)"; - } + define('EXPECTED_CONFIG_VERSION', 24); + define('SCHEMA_VERSION', 86); - if (GENERATED_CONFIG_CHECK != EXPECTED_CONFIG_VERSION) { - $err_msg = "config: your sanity_config.php is outdated, please recreate it using ./utils/regen_config_checks.sh"; - } + require_once "config.php"; + require_once "sanity_config.php"; - foreach ($requred_defines as $d) { - if (!defined($d)) { - $err_msg = "config: required constant $d is not defined. Please check config.php"; + if (CONFIG_VERSION != EXPECTED_CONFIG_VERSION) { + $err_msg = "Configuration file (config.php) has incorrect version. Update it with new options from config.php-dist and set CONFIG_VERSION to the correct value."; } - } - - if (defined('RSS_BACKEND_TYPE')) { - print "Fatal error: RSS_BACKEND_TYPE is deprecated. Please remove this - option from config.php\n"; - exit; - } - - if (file_exists("xml-export.php") || file_exists("xml-import.php")) { - print "Fatal Error: XML Import/Export tools (xml-export.php - and xml-import.php) could be used maliciously. Please remove them - from your TT-RSS instance.\n"; - exit; - } - - if (SINGLE_USER_MODE && DAEMON_UPDATE_LOGIN_LIMIT > 0) { - print "Fatal Error: Please set DAEMON_UPDATE_LOGIN_LIMIT - to 0 in single user mode.\n"; - exit; - } - if (!defined('SESSION_EXPIRE_TIME')) { - $err_msg = "config: SESSION_EXPIRE_TIME is undefined"; - } + $purifier_cache_dir = CACHE_DIR . "/htmlpurifier"; - if (SESSION_EXPIRE_TIME < 60) { - $err_msg = "config: SESSION_EXPIRE_TIME is too low (less than 60)"; - } + if (!is_writable($purifier_cache_dir)) { + $err_msg = "HTMLPurifier cache directory should be writable by anyone (chmod -R 777 $purifier_cache_dir)"; + } - if (SESSION_EXPIRE_TIME < SESSION_COOKIE_LIFETIME) { - $err_msg = "config: SESSION_EXPIRE_TIME should be greater or equal to" . - "SESSION_COOKIE_LIFETIME"; - } + if (GENERATED_CONFIG_CHECK != EXPECTED_CONFIG_VERSION) { + $err_msg = "Configuration option checker sanity_config.php is outdated, please recreate it using ./utils/regen_config_checks.sh"; + } -/* if (defined('DISABLE_SESSIONS')) { - $err_msg = "config: you have enabled DISABLE_SESSIONS. Please disable this option."; -} */ + foreach ($requred_defines as $d) { + if (!defined($d)) { + $err_msg = "Required configuration file parameter $d is not defined in config.php. You might need to copy it from config.php-dist."; + } + } - if (DATABASE_BACKED_SESSIONS && SINGLE_USER_MODE) { - $err_msg = "config: DATABASE_BACKED_SESSIONS is incompatible with SINGLE_USER_MODE"; - } + if (SESSION_EXPIRE_TIME < 60) { + $err_msg = "SESSION_EXPIRE_TIME set in config.php is too low, please set it to an integer value >= 60"; + } - if (DATABASE_BACKED_SESSIONS && DB_TYPE == "mysql") { - $err_msg = "config: DATABASE_BACKED_SESSIONS are currently broken with MySQL"; - } + if (SESSION_EXPIRE_TIME < SESSION_COOKIE_LIFETIME) { + $err_msg = "SESSION_EXPIRE_TIME set in config.php should be >= to SESSION_COOKIE_LIFETIME"; + } - if (SINGLE_USER_MODE) { - $link = db_connect(DB_HOST, DB_USER, DB_PASS, DB_NAME); + if (SINGLE_USER_MODE) { + $link = db_connect(DB_HOST, DB_USER, DB_PASS, DB_NAME); - if ($link) { - $result = db_query($link, "SELECT id FROM ttrss_users WHERE id = 1"); + if ($link) { + $result = db_query($link, "SELECT id FROM ttrss_users WHERE id = 1"); - if (db_num_rows($result) != 1) { - $err_msg = "config: SINGLE_USER_MODE is enabled but default admin account (UID=1) is not found."; + if (db_num_rows($result) != 1) { + $err_msg = "SINGLE_USER_MODE is enabled in config.php but default admin account is not found."; + } } } - } - - if (defined('MAIL_FROM')) { - $err_msg = "config: MAIL_FROM has been split into DIGEST_FROM_NAME and DIGEST_FROM_ADDRESS"; - } - - if (!defined('COUNTERS_MAX_AGE')) { - $err_msg = "config: option COUNTERS_MAX_AGE expected, but not defined"; - } - if (defined('DAEMON_REFRESH_ONLY')) { - $err_msg = "config: option DAEMON_REFRESH_ONLY is obsolete. Please remove this option and read about other ways to update feeds on the wiki."; + if (SELF_URL_PATH == "http://yourserver/tt-rss/") { + if ($_SERVER['HTTP_REFERER']) { + $err_msg = "Please set SELF_URL_PATH to the correct value for your server (possible value: " . $_SERVER['HTTP_REFERER'] . ")"; + } else { + $err_msg = "Please set SELF_URL_PATH to the correct value for your server."; + } + } - } + if (!is_writable(ICONS_DIR)) { + $err_msg = "ICONS_DIR defined in config.php is not writable (chmod -R 777 ".ICONS_DIR.").\n"; + } - if (defined('ENABLE_SIMPLEPIE')) { - $err_msg = "config: ENABLE_SIMPLEPIE is obsolete and replaced with DEFAULT_UPDATE_METHOD. Please adjust your config.php."; - } + if (ini_get("open_basedir")) { + $err_msg = "PHP configuration option open_basedir is not supported. Please disable this in PHP settings file (php.ini)."; + } - if (!defined('DEFAULT_UPDATE_METHOD') || (DEFAULT_UPDATE_METHOD != 0 && - DEFAULT_UPDATE_METHOD != 1)) { - $err_msg = "config: DEFAULT_UPDATE_METHOD should be either 0 or 1."; - } + if (!function_exists("curl_init") && !ini_get("allow_url_fopen")) { + $err_msg = "PHP configuration option allow_url_fopen is disabled, and CURL functions are not present. Either enable allow_url_fopen or install PHP extension for CURL."; + } - if (SELF_URL_PATH == "http://yourserver/tt-rss/") { - $err_msg = "config: please set SELF_URL_PATH to the correct value."; - } + if (!function_exists("json_encode")) { + $err_msg = "PHP support for JSON is required, but was not found."; + } - if (!is_writable(ICONS_DIR)) { - $err_msg = "config: your ICONS_DIR (" . ICONS_DIR . ") is not writable.\n"; - } + if (DB_TYPE == "mysql" && !function_exists("mysql_connect")) { + $err_msg = "PHP support for MySQL is required for configured DB_TYPE in config.php."; + } - if (ini_get("open_basedir")) { - $err_msg = "php.ini: open_basedir is not supported."; - } + if (DB_TYPE == "pgsql" && !function_exists("pg_connect")) { + $err_msg = "PHP support for PostgreSQL is required for configured DB_TYPE in config.php"; + } - if (!function_exists("curl_init") && !ini_get("allow_url_fopen")) { - $err_msg = "php.ini: either allow_url_fopen or CURL needs to be enabled."; - } + if (!function_exists("mb_strlen")) { + $err_msg = "PHP support for mbstring functions is required, but was not found."; + } - if (!function_exists("json_encode")) { - $err_msg = "PHP: json functions not found."; - } + if (!function_exists("ctype_lower")) { + $err_msg = "PHP support for ctype functions are required by HTMLPurifier."; + } - if (DB_TYPE == "mysql" && !function_exists("mysql_connect")) { - $err_msg = "PHP: MySQL functions not found."; - } + if (ini_get("safe_mode")) { + $err_msg = "PHP safe mode setting is not supported."; + } - if (DB_TYPE == "pgsql" && !function_exists("pg_connect")) { - $err_msg = "PHP: PostgreSQL functions not found."; - } + if ((PUBSUBHUBBUB_HUB || PUBSUBHUBBUB_ENABLED) && !function_exists("curl_init")) { + $err_msg = "PHP support for CURL is required for PubSubHubbub."; + } - if (!function_exists("mb_strlen")) { - $err_msg = "PHP: mbstring functions not found."; + if (!class_exists("DOMDocument")) { + $err_msg = "PHP support for DOMDocument is required, but was not found."; + } } - if (!function_exists("ctype_lower")) { - $err_msg = "PHP: ctype functions not found (required for HTMLPurifier)."; - } + if ($err_msg) { ?> + + + Fatal error + + + - if (ini_get("safe_mode")) { - $err_msg = "php.ini: Safe mode is not supported. If you wish to continue, remove this test from sanity_check.php and proceeed at your own risk. Please note that your bug reports will not be accepted or reviewed."; - } + - if ((PUBSUBHUBBUB_HUB || PUBSUBHUBBUB_ENABLED) && !function_exists("curl_init")) { - $err_msg = "CURL is required for PubSubHubbub support."; - } +

Fatal error

- if (!class_exists("DOMDocument")) { - $err_msg = "PHP: DOMDocument extension not found."; - } +

Tiny Tiny RSS was unable to initialize properly. This usually means a misconfiguration or an incomplete upgrade. Please fix + the error indicated by the following message:

- if (SELF_URL_PATH == "http://local.host/tt-rss") { - $err_msg = "config: please set SELF_URL_PATH to the correct value"; - } +

You might want to check tt-rss wiki or the + forums for more information. Please search the forums before creating new topic + for your question.

- if (!ISCONFIGURED) { - $err_msg = "config: please read config.php completely."; - } + + + + - if ($err_msg) { - print "Fatal Error: $err_msg\n"; - exit; + diff --git a/include/sanity_config.php b/include/sanity_config.php index 51c9d52b..5ed49de5 100644 --- a/include/sanity_config.php +++ b/include/sanity_config.php @@ -1,3 +1,3 @@ - + diff --git a/include/sessions.php b/include/sessions.php index 8588f580..5aeda62f 100644 --- a/include/sessions.php +++ b/include/sessions.php @@ -96,7 +96,7 @@ db_query($session_connection, $query); } - if (DATABASE_BACKED_SESSIONS) { + if (!SINGLE_USER_MODE && DB_TYPE == "pgsql") { session_set_save_handler("ttrss_open", "ttrss_close", "ttrss_read", "ttrss_write", "ttrss_destroy", "ttrss_gc"); diff --git a/register.php b/register.php index 33a6628f..d63fc251 100644 --- a/register.php +++ b/register.php @@ -311,15 +311,15 @@ $mail->CharSet = "UTF-8"; - $mail->From = DIGEST_FROM_ADDRESS; - $mail->FromName = DIGEST_FROM_NAME; + $mail->From = SMTP_FROM_ADDRESS; + $mail->FromName = SMTP_FROM_NAME; $mail->AddAddress($email); - if (DIGEST_SMTP_HOST) { - $mail->Host = DIGEST_SMTP_HOST; + if (SMTP_HOST) { + $mail->Host = SMTP_HOST; $mail->Mailer = "smtp"; - $mail->Username = DIGEST_SMTP_LOGIN; - $mail->Password = DIGEST_SMTP_PASSWORD; + $mail->Username = SMTP_LOGIN; + $mail->Password = SMTP_PASSWORD; } // $mail->IsHTML(true); @@ -345,15 +345,15 @@ $mail->CharSet = "UTF-8"; - $mail->From = DIGEST_FROM_ADDRESS; - $mail->FromName = DIGEST_FROM_NAME; + $mail->From = SMTP_FROM_ADDRESS; + $mail->FromName = SMTP_FROM_NAME; $mail->AddAddress(REG_NOTIFY_ADDRESS); - if (DIGEST_SMTP_HOST) { - $mail->Host = DIGEST_SMTP_HOST; + if (SMTP_HOST) { + $mail->Host = SMTP_HOST; $mail->Mailer = "smtp"; - $mail->Username = DIGEST_SMTP_LOGIN; - $mail->Password = DIGEST_SMTP_PASSWORD; + $mail->Username = SMTP_LOGIN; + $mail->Password = SMTP_PASSWORD; } // $mail->IsHTML(true); diff --git a/utility.css b/utility.css index 0b80789c..14357b4d 100644 --- a/utility.css +++ b/utility.css @@ -60,7 +60,6 @@ div.error { padding : 5px; margin : 0px 0px 5px 0px; font-size : 9pt; - width : 50%; } h1 { -- 2.39.2