From b0a296e2e1dd1b173d9cab8c82158ac6bfb86f9e Mon Sep 17 00:00:00 2001 From: Morris Jobke Date: Tue, 26 Jun 2018 12:15:09 +0200 Subject: [PATCH] Do not use HTTP code OC_Response constants anymore Signed-off-by: Morris Jobke --- index.php | 8 ++++---- lib/base.php | 2 +- lib/private/legacy/response.php | 9 --------- public.php | 9 ++++----- remote.php | 14 +++++++------- 5 files changed, 16 insertions(+), 26 deletions(-) diff --git a/index.php b/index.php index f0e763f797..4b5991a3ad 100644 --- a/index.php +++ b/index.php @@ -45,10 +45,10 @@ try { \OC::$server->getLogger()->logException($ex, array('app' => 'index')); //show the user a detailed error page - OC_Template::printExceptionErrorPage($ex, \OC_Response::STATUS_SERVICE_UNAVAILABLE); + OC_Template::printExceptionErrorPage($ex, 503); } catch (\OC\HintException $ex) { try { - OC_Template::printErrorPage($ex->getMessage(), $ex->getHint(), OC_Response::STATUS_SERVICE_UNAVAILABLE); + OC_Template::printErrorPage($ex->getMessage(), $ex->getHint(), 503); } catch (Exception $ex2) { \OC::$server->getLogger()->logException($ex, array('app' => 'index')); \OC::$server->getLogger()->logException($ex2, array('app' => 'index')); @@ -57,7 +57,7 @@ try { OC_Template::printExceptionErrorPage($ex, 500); } } catch (\OC\User\LoginException $ex) { - OC_Template::printErrorPage($ex->getMessage(), $ex->getMessage(), OC_Response::STATUS_FORBIDDEN); + OC_Template::printErrorPage($ex->getMessage(), $ex->getMessage(), 403); } catch (Exception $ex) { \OC::$server->getLogger()->logException($ex, array('app' => 'index')); @@ -76,5 +76,5 @@ try { throw $e; } - OC_Template::printExceptionErrorPage($ex, \OC_Response::STATUS_INTERNAL_SERVER_ERROR); + OC_Template::printExceptionErrorPage($ex, 500); } diff --git a/lib/base.php b/lib/base.php index 598e3ed575..8ca4a371af 100644 --- a/lib/base.php +++ b/lib/base.php @@ -432,7 +432,7 @@ class OC { } catch (Exception $e) { \OC::$server->getLogger()->logException($e, ['app' => 'base']); //show the user a detailed error page - OC_Template::printExceptionErrorPage($e, \OC_Response::STATUS_INTERNAL_SERVER_ERROR); + OC_Template::printExceptionErrorPage($e, 500); die(); } diff --git a/lib/private/legacy/response.php b/lib/private/legacy/response.php index 2fc7b934d8..93023f61e9 100644 --- a/lib/private/legacy/response.php +++ b/lib/private/legacy/response.php @@ -31,15 +31,6 @@ */ class OC_Response { - const STATUS_FOUND = 302; - const STATUS_NOT_MODIFIED = 304; - const STATUS_TEMPORARY_REDIRECT = 307; - const STATUS_BAD_REQUEST = 400; - const STATUS_FORBIDDEN = 403; - const STATUS_NOT_FOUND = 404; - const STATUS_INTERNAL_SERVER_ERROR = 500; - const STATUS_SERVICE_UNAVAILABLE = 503; - /** * Sets the content disposition header (with possible workarounds) * @param string $filename file name diff --git a/public.php b/public.php index 5d46f4970d..d50f49e253 100644 --- a/public.php +++ b/public.php @@ -36,7 +36,7 @@ try { if (\OCP\Util::needUpgrade()) { // since the behavior of apps or remotes are unpredictable during // an upgrade, return a 503 directly - OC_Template::printErrorPage('Service unavailable', '', OC_Response::STATUS_SERVICE_UNAVAILABLE); + OC_Template::printErrorPage('Service unavailable', '', 503); exit; } @@ -78,10 +78,9 @@ try { require_once OC_App::getAppPath($app) . '/' . $parts[1]; } catch (Exception $ex) { + $status = 500; if ($ex instanceof \OC\ServiceUnavailableException) { - $status = OC_Response::STATUS_SERVICE_UNAVAILABLE; - } else { - $status = OC_Response::STATUS_INTERNAL_SERVER_ERROR; + $status = 503; } //show the user a detailed error page \OC::$server->getLogger()->logException($ex, ['app' => 'public']); @@ -89,5 +88,5 @@ try { } catch (Error $ex) { //show the user a detailed error page \OC::$server->getLogger()->logException($ex, ['app' => 'public']); - OC_Template::printExceptionErrorPage($ex, OC_Response::STATUS_INTERNAL_SERVER_ERROR); + OC_Template::printExceptionErrorPage($ex, 500); } diff --git a/remote.php b/remote.php index d90bb2d8ee..11940b025d 100644 --- a/remote.php +++ b/remote.php @@ -59,9 +59,9 @@ function handleException($e) { $server->on('beforeMethod', function () use ($e) { if ($e instanceof RemoteException) { switch ($e->getCode()) { - case OC_Response::STATUS_SERVICE_UNAVAILABLE: + case 503: throw new ServiceUnavailable($e->getMessage()); - case OC_Response::STATUS_NOT_FOUND: + case 404: throw new \Sabre\DAV\Exception\NotFound($e->getMessage()); } } @@ -71,9 +71,9 @@ function handleException($e) { }); $server->exec(); } else { - $statusCode = OC_Response::STATUS_INTERNAL_SERVER_ERROR; + $statusCode = 500; if ($e instanceof \OC\ServiceUnavailableException ) { - $statusCode = OC_Response::STATUS_SERVICE_UNAVAILABLE; + $statusCode = 503; } if ($e instanceof RemoteException) { // we shall not log on RemoteException @@ -118,13 +118,13 @@ try { if (\OCP\Util::needUpgrade()) { // since the behavior of apps or remotes are unpredictable during // an upgrade, return a 503 directly - throw new RemoteException('Service unavailable', OC_Response::STATUS_SERVICE_UNAVAILABLE); + throw new RemoteException('Service unavailable', 503); } $request = \OC::$server->getRequest(); $pathInfo = $request->getPathInfo(); if ($pathInfo === false || $pathInfo === '') { - throw new RemoteException('Path not found', OC_Response::STATUS_NOT_FOUND); + throw new RemoteException('Path not found', 404); } if (!$pos = strpos($pathInfo, '/', 1)) { $pos = strlen($pathInfo); @@ -134,7 +134,7 @@ try { $file = resolveService($service); if(is_null($file)) { - throw new RemoteException('Path not found', OC_Response::STATUS_NOT_FOUND); + throw new RemoteException('Path not found', 404); } $file=ltrim($file, '/');