From 703f3551dc2c9caaebbcd2e9d9377f314dfd28ea Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Thomas=20M=C3=BCller?= Date: Mon, 8 Feb 2016 17:49:25 +0100 Subject: [PATCH] Only set the header if the node exists and in case the request is a GET --- apps/dav/lib/server.php | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/apps/dav/lib/server.php b/apps/dav/lib/server.php index 0c25890b66..f5f1875a48 100644 --- a/apps/dav/lib/server.php +++ b/apps/dav/lib/server.php @@ -110,10 +110,15 @@ class Server { // Serve all files with an Content-Disposition of type "attachment" $this->server->on('beforeMethod', function (RequestInterface $requestInterface, ResponseInterface $responseInterface) { - $node = $this->server->tree->getNodeForPath($requestInterface->getPath()); - if (($node instanceof IFile)) { - $responseInterface->addHeader('Content-Disposition', 'attachment'); + if ($requestInterface->getMethod() === 'GET') { + $path = $requestInterface->getPath(); + if ($this->server->tree->nodeExists($path)) { + $node = $this->server->tree->getNodeForPath($path); + if (($node instanceof IFile)) { + $responseInterface->addHeader('Content-Disposition', 'attachment'); + } } + } }); // wait with registering these until auth is handled and the filesystem is setup