diff --git a/cron.php b/cron.php index c623689722..8c02536744 100644 --- a/cron.php +++ b/cron.php @@ -41,6 +41,11 @@ try { exit; } + if (\OC::$server->getSystemConfig()->getValue('singleuser', false)) { + \OCP\Util::writeLog('cron', 'We are in admin only mode, skipping cron', \OCP\Util::DEBUG); + exit; + } + // load all apps to get all api routes properly setup OC_App::loadApps(); diff --git a/lib/base.php b/lib/base.php index 042419eff1..74c1e8c0f4 100644 --- a/lib/base.php +++ b/lib/base.php @@ -294,27 +294,37 @@ class OC { header('Retry-After: 120'); // render error page - $tmpl = new OC_Template('', 'update.user', 'guest'); + $template = new OC_Template('', 'update.user', 'guest'); OC_Util::addscript('maintenance-check'); - $tmpl->printPage(); + $template->printPage(); die(); } } - public static function checkSingleUserMode() { - $user = OC_User::getUserSession()->getUser(); - $group = OC_Group::getManager()->get('admin'); - if ($user && \OC::$server->getSystemConfig()->getValue('singleuser', false) && !$group->inGroup($user)) { - // send http status 503 - header('HTTP/1.1 503 Service Temporarily Unavailable'); - header('Status: 503 Service Temporarily Unavailable'); - header('Retry-After: 120'); - - // render error page - $tmpl = new OC_Template('', 'singleuser.user', 'guest'); - $tmpl->printPage(); - die(); + public static function checkSingleUserMode($lockIfNoUserLoggedIn = false) { + if (!\OC::$server->getSystemConfig()->getValue('singleuser', false)) { + return; } + $user = OC_User::getUserSession()->getUser(); + if ($user) { + $group = \OC::$server->getGroupManager()->get('admin'); + if ($group->inGroup($user)) { + return; + } + } else { + if(!$lockIfNoUserLoggedIn) { + return; + } + } + // send http status 503 + header('HTTP/1.1 503 Service Temporarily Unavailable'); + header('Status: 503 Service Temporarily Unavailable'); + header('Retry-After: 120'); + + // render error page + $template = new OC_Template('', 'singleuser.user', 'guest'); + $template->printPage(); + die(); } /** diff --git a/lib/private/connector/sabre/maintenanceplugin.php b/lib/private/connector/sabre/maintenanceplugin.php index 91c48a45d3..c4c9a5340b 100644 --- a/lib/private/connector/sabre/maintenanceplugin.php +++ b/lib/private/connector/sabre/maintenanceplugin.php @@ -60,6 +60,9 @@ class MaintenancePlugin extends \Sabre\DAV\ServerPlugin * @return bool */ public function checkMaintenanceMode() { + if (\OC::$server->getSystemConfig()->getValue('singleuser', false)) { + throw new \Sabre\DAV\Exception\ServiceUnavailable(); + } if (\OC_Config::getValue('maintenance', false)) { throw new \Sabre\DAV\Exception\ServiceUnavailable(); } diff --git a/ocs/v1.php b/ocs/v1.php index 398a128c64..8002ca602e 100644 --- a/ocs/v1.php +++ b/ocs/v1.php @@ -27,7 +27,9 @@ require_once '../lib/base.php'; -if (\OCP\Util::needUpgrade() || \OC::$server->getSystemConfig()->getValue('maintenance', false)) { +if (\OCP\Util::needUpgrade() + || \OC::$server->getSystemConfig()->getValue('maintenance', false) + || \OC::$server->getSystemConfig()->getValue('singleuser', false)) { // since the behavior of apps or remotes are unpredictable during // an upgrade, return a 503 directly OC_Response::setStatus(OC_Response::STATUS_SERVICE_UNAVAILABLE); diff --git a/public.php b/public.php index 8f3bc8e6ae..f390166d06 100644 --- a/public.php +++ b/public.php @@ -37,7 +37,7 @@ try { } OC::checkMaintenanceMode(); - OC::checkSingleUserMode(); + OC::checkSingleUserMode(true); $request = \OC::$server->getRequest(); $pathInfo = $request->getPathInfo();