diff options
author | Morris Jobke <hey@morrisjobke.de> | 2020-10-26 16:22:27 +0100 |
---|---|---|
committer | Morris Jobke <hey@morrisjobke.de> | 2020-10-26 16:22:27 +0100 |
commit | 0390a90d70104c2b3237feb05c3398f8ea1612c9 (patch) | |
tree | d5d8549a204094fcd1420b640e81804283000c0f /apps | |
parent | ca1dc0009dfde447c90db826465defbb156966da (diff) | |
download | nextcloud-server-0390a90d70104c2b3237feb05c3398f8ea1612c9.tar.gz nextcloud-server-0390a90d70104c2b3237feb05c3398f8ea1612c9.zip |
Return correct error message on share download without read permission
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Diffstat (limited to 'apps')
-rw-r--r-- | apps/files_sharing/lib/Controller/ShareController.php | 2 | ||||
-rw-r--r-- | apps/files_sharing/tests/Controller/ShareControllerTest.php | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/apps/files_sharing/lib/Controller/ShareController.php b/apps/files_sharing/lib/Controller/ShareController.php index fdf69abd9ac..3e514b3a74d 100644 --- a/apps/files_sharing/lib/Controller/ShareController.php +++ b/apps/files_sharing/lib/Controller/ShareController.php @@ -556,7 +556,7 @@ class ShareController extends AuthPublicShareController { $share = $this->shareManager->getShareByToken($token); if (!($share->getPermissions() & \OCP\Constants::PERMISSION_READ)) { - return new \OCP\AppFramework\Http\DataResponse('Share is read-only'); + return new \OCP\AppFramework\Http\DataResponse('Share has no read permission'); } $files_list = null; diff --git a/apps/files_sharing/tests/Controller/ShareControllerTest.php b/apps/files_sharing/tests/Controller/ShareControllerTest.php index 33dff13cafe..270f38a1148 100644 --- a/apps/files_sharing/tests/Controller/ShareControllerTest.php +++ b/apps/files_sharing/tests/Controller/ShareControllerTest.php @@ -840,7 +840,7 @@ class ShareControllerTest extends \Test\TestCase { // Test with a password protected share and no authentication $response = $this->shareController->downloadShare('validtoken'); - $expectedResponse = new DataResponse('Share is read-only'); + $expectedResponse = new DataResponse('Share has no read permission'); $this->assertEquals($expectedResponse, $response); } |