]> source.dussan.org Git - nextcloud-server.git/commitdiff
Fix invalid usage of \Exception::getResult 21432/head
authorChristoph Wurst <christoph@winzerhof-wurst.at>
Tue, 16 Jun 2020 13:37:20 +0000 (15:37 +0200)
committerChristoph Wurst <christoph@winzerhof-wurst.at>
Tue, 16 Jun 2020 13:37:20 +0000 (15:37 +0200)
Only OCS exceptions have a getResult method. Any other exception will
cause another error due to this invalid method call.

This splits the catch into a specific one for OCS and then a generic one
for anything else that can't be handled.

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
ocs/v1.php

index 983bfd5f77da1118dcce4ddc61ad5f03cdd57fc8..0c24aa814ebbc95a259f57027cc5b1f989832d0b 100644 (file)
@@ -67,9 +67,15 @@ try {
        OC_API::setContentType();
        http_response_code(405);
        exit();
-} catch (Exception $ex) {
+} catch (\OC\OCS\Exception $ex) {
        OC_API::respond($ex->getResult(), OC_API::requestedFormat());
        exit();
+} catch (Throwable $ex) {
+       OC::$server->getLogger()->logException($ex);
+
+       OC_API::setContentType();
+       http_response_code(500);
+       exit();
 }
 
 /*