From: Vincent Petry <pvince81@owncloud.com>
Date: Fri, 30 Jan 2015 11:02:23 +0000 (+0100)
Subject: Fix DAV exceptions, some will still return false now
X-Git-Tag: v8.0.0RC2~8^2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=5ade2854ee0f47e6004fbfbaa59a152c870bf6e0;p=nextcloud-server.git

Fix DAV exceptions, some will still return false now

Since the Storage interface isn't ready to work directly with exceptions
like Forbidden and NotFound, the DAV storage has been adapted to still
return false when expected.
---

diff --git a/lib/private/files/storage/dav.php b/lib/private/files/storage/dav.php
index 2d4916df08a..4f7b3ff8940 100644
--- a/lib/private/files/storage/dav.php
+++ b/lib/private/files/storage/dav.php
@@ -128,6 +128,7 @@ class DAV extends \OC\Files\Storage\Common {
 			return false;
 		} catch (\Sabre\DAV\Exception $e) {
 			$this->convertSabreException($e);
+			return false;
 		} catch (\Exception $e) {
 			// TODO: log for now, but in the future need to wrap/rethrow exception
 			\OCP\Util::writeLog('files_external', $e->getMessage(), \OCP\Util::ERROR);
@@ -149,6 +150,7 @@ class DAV extends \OC\Files\Storage\Common {
 			return false;
 		} catch (\Sabre\DAV\Exception $e) {
 			$this->convertSabreException($e);
+			return false;
 		} catch (\Exception $e) {
 			// TODO: log for now, but in the future need to wrap/rethrow exception
 			\OCP\Util::writeLog('files_external', $e->getMessage(), \OCP\Util::ERROR);
@@ -166,6 +168,7 @@ class DAV extends \OC\Files\Storage\Common {
 			return false;
 		} catch (\Sabre\DAV\Exception $e) {
 			$this->convertSabreException($e);
+			return false;
 		} catch (\Exception $e) {
 			// TODO: log for now, but in the future need to wrap/rethrow exception
 			\OCP\Util::writeLog('files_external', $e->getMessage(), \OCP\Util::ERROR);
@@ -284,6 +287,7 @@ class DAV extends \OC\Files\Storage\Common {
 				return false;
 			} catch (\Sabre\DAV\Exception $e) {
 				$this->convertSabreException($e);
+				return false;
 			} catch (\Exception $e) {
 				// TODO: log for now, but in the future need to wrap/rethrow exception
 				\OCP\Util::writeLog('files_external', $e->getMessage(), \OCP\Util::ERROR);
@@ -337,6 +341,7 @@ class DAV extends \OC\Files\Storage\Common {
 			return true;
 		} catch (\Sabre\DAV\Exception $e) {
 			$this->convertSabreException($e);
+			return false;
 		} catch (\Exception $e) {
 			// TODO: log for now, but in the future need to wrap/rethrow exception
 			\OCP\Util::writeLog('files_external', $e->getMessage(), \OCP\Util::ERROR);
@@ -354,6 +359,7 @@ class DAV extends \OC\Files\Storage\Common {
 			return true;
 		} catch (\Sabre\DAV\Exception $e) {
 			$this->convertSabreException($e);
+			return false;
 		} catch (\Exception $e) {
 			// TODO: log for now, but in the future need to wrap/rethrow exception
 			\OCP\Util::writeLog('files_external', $e->getMessage(), \OCP\Util::ERROR);
@@ -374,6 +380,7 @@ class DAV extends \OC\Files\Storage\Common {
 			return array();
 		} catch (\Sabre\DAV\Exception $e) {
 			$this->convertSabreException($e);
+			return false;
 		} catch (\Exception $e) {
 			// TODO: log for now, but in the future need to wrap/rethrow exception
 			\OCP\Util::writeLog('files_external', $e->getMessage(), \OCP\Util::ERROR);
@@ -398,8 +405,11 @@ class DAV extends \OC\Files\Storage\Common {
 			} else {
 				return false;
 			}
+		} catch (Exception\NotFound $e) {
+			return false;
 		} catch (\Sabre\DAV\Exception $e) {
 			$this->convertSabreException($e);
+			return false;
 		} catch (\Exception $e) {
 			// TODO: log for now, but in the future need to wrap/rethrow exception
 			\OCP\Util::writeLog('files_external', $e->getMessage(), \OCP\Util::ERROR);
@@ -440,8 +450,16 @@ class DAV extends \OC\Files\Storage\Common {
 		try {
 			$response = $this->client->request($method, $this->encodePath($path), $body);
 			return $response['statusCode'] == $expected;
+		} catch (Exception\NotFound $e) {
+			if ($method === 'DELETE') {
+				return false;
+			}
+
+			$this->convertSabreException($e);
+			return false;
 		} catch (\Sabre\DAV\Exception $e) {
 			$this->convertSabreException($e);
+			return false;
 		} catch (\Exception $e) {
 			// TODO: log for now, but in the future need to wrap/rethrow exception
 			\OCP\Util::writeLog('files_external', $e->getMessage(), \OCP\Util::ERROR);
@@ -549,6 +567,7 @@ class DAV extends \OC\Files\Storage\Common {
 			return false;
 		} catch (Exception $e) {
 			$this->convertSabreException($e);
+			return false;
 		}
 	}
 
@@ -567,6 +586,9 @@ class DAV extends \OC\Files\Storage\Common {
 		if ($e instanceof \Sabre\DAV\Exception\NotAuthenticated) {
 			// either password was changed or was invalid all along
 			throw new StorageInvalidException(get_class($e).': '.$e->getMessage());
+		} else if ($e instanceof \Sabre\DAV\Exception\MethodNotAllowed) {
+			// ignore exception, false will be returned
+			return;
 		}
 
 		throw new StorageNotAvailableException(get_class($e).': '.$e->getMessage());