Merge pull request #20073 from owncloud/files-should-add-download-disposition

Serve files with an attachment disposition for new DAV endpoint
This commit is contained in:
Thomas Müller 2016-02-10 14:35:50 +01:00
commit 159a0eb597
1 changed files with 17 additions and 0 deletions

View File

@ -30,6 +30,10 @@ use OCA\DAV\Files\CustomPropertiesBackend;
use OCP\IRequest;
use OCP\SabrePluginEvent;
use Sabre\DAV\Auth\Plugin;
use Sabre\DAV\IFile;
use Sabre\HTTP\RequestInterface;
use Sabre\HTTP\ResponseInterface;
use Sabre\HTTP\Util;
class Server {
@ -104,6 +108,19 @@ class Server {
$this->server->addPlugin(new \OCA\DAV\Connector\Sabre\FakeLockerPlugin());
}
// Serve all files with an Content-Disposition of type "attachment"
$this->server->on('beforeMethod', function (RequestInterface $requestInterface, ResponseInterface $responseInterface) {
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
$this->server->on('beforeMethod', function () {
// custom properties plugin must be the last one